1715112 - [css-overflow-4] Implement "scrollbar-gutter" css property

Status: RESOLVED FIXED

(Core :: Layout: Scrolling and Overflow, enhancement)

Description

[Daniel Holbert [:dholbert]]

https://drafts.csswg.org/css-overflow-4/#scrollbar-gutter-property

Currently it just takes:

Value: 	auto | stable && mirror?

"auto" is the existing behavior.
"stable" sets aside space for a scrollbar gutter in (non-overlay) scrollable elements that otherwise wouldn't be showing a scrollbar (e.g. not-overflowed "overflow:auto", and also overflow:hidden)

"mirror" makes us reserve some scrollbar space on the opposite side as well.

Comment 1

[Ting-Yu Lin [:TYLin] (PST, UTC-8)]

Per https://github.com/w3c/csswg-drafts/issues/6349#issuecomment-880053625, the syntax is changed to

Value: 	auto | stable && both-edges?

Comment 2

[Emilio Cobos Álvarez (:emilio)]

Happy to mentor this if someone wants to give it a shot :)

Comment 3

[Ting-Yu Lin [:TYLin] (PST, UTC-8)]

Attachment: Bug 1715112 Part 1 - Implement the rendering of scrollbar-gutter.

WIP: Some wpt tests still fail.

Depends on D132663

Comment 4

[Ting-Yu Lin [:TYLin] (PST, UTC-8)]

Attachment: Bug 1715112 Part 2 - Add tests variants for vertical writing-mode and rtl direction.

Depends on D132664

Comment 5

[Ting-Yu Lin [:TYLin] (PST, UTC-8)]

Attachment: Bug 1715112 Part 3 - Enable CSS scrollbar-gutter by default.

Depends on D133695

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9b7d73b6d81e
Part 1 - Implement the rendering of scrollbar-gutter. r=emilio
https://hg.mozilla.org/integration/autoland/rev/b7cbd5b59fc5
Part 2 - Add tests variants for vertical writing-mode and rtl direction. r=emilio
https://hg.mozilla.org/integration/autoland/rev/6b91412c20e3
Part 3 - Enable CSS scrollbar-gutter by default. r=emilio

Comment 7

[Ting-Yu Lin [:TYLin] (PST, UTC-8)]

"Intent to ship" sent. https://groups.google.com/a/mozilla.org/g/dev-platform/c/pAh2tDhBFOU/m/wQBctGKCCwAJ

Comment 8

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/32098 for changes under testing/web-platform/tests

Comment 9

[Marian-Vasile Laza]

Backed out 3 changesets (Bug 1715112) for causing xpcshell failures on test_css-properties-db.js.
Backout link](https://hg.mozilla.org/integration/autoland/rev/31ae39c96ef8e5ab9da2dd2d209e64ff531abed4)
Push with failures
Failure Log

Comment 10

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR was closed without merging

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/83044c2e662d
Part 1 - Implement the rendering of scrollbar-gutter. r=emilio
https://hg.mozilla.org/integration/autoland/rev/24b42e6ae0f9
Part 2 - Add tests variants for vertical writing-mode and rtl direction. r=emilio
https://hg.mozilla.org/integration/autoland/rev/4bc795a37d13
Part 3 - Enable CSS scrollbar-gutter by default. r=emilio

Comment 12

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/83044c2e662d
https://hg.mozilla.org/mozilla-central/rev/24b42e6ae0f9
https://hg.mozilla.org/mozilla-central/rev/4bc795a37d13

Comment 13

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot