refactor: make grid scroller inherit border radius in base styles

[web-padawan]

Description

Part of #10417

Updated grid to just inherit border-radius on the scroller element.

Type of change

• Refactor

[jouni]

I’m now thinking we should remove the --_border-width and --_border-radius properties, and add overflow: hidden on the host. I don't remember anymore why I wanted to avoid that. Then we don't need to update the screenshots.

We could also simplify the no-border variant implementation, avoid the :not() selector and just override border and border-radius to zero for that variant.

[web-padawan]

@jouni updated, please check if I got it right. Setting overflow: hidden affected the dragover screenshot.

UPD: some tests also failed because of the change, we should double check that as well.

[jouni]

please check if I got it right

Yeah, this is what I meant.

Setting overflow: hidden affected the dragover screenshot.

That’s weird. That's set using outline on the host, so it should still be visible.

UPD: ah, it relied on the var(--_border-width) property, so it’s now not inset properly on top of the host border.

[web-padawan]

@jouni Updated the outline styles. Regarding unit test - I had to set overflow: visible for the all-rows-visible to prevent regression for the issue fixed in #8535.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud