Closed Bug 1689306 Opened 5 years ago Closed 2 years ago

[wpt-sync] Sync PR 27364 - Initial implementation of scrollbar-width

Categories

(Core :: Layout: Scrolling and Overflow, task, P4)

task

Tracking

()

RESOLVED INVALID

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 27364 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/27364
Details from upstream follow.

b'Felipe Erias <felipeerias@igalia.com>' wrote:

Initial implementation of scrollbar-width

ScrollbarTheme::ScrollbarThickness() takes scrollbar-width into
account when calculating the thickness.

Scrollbar themes return a thickness of 0 if scrollbar-width is "none"
and the same value for both "auto" and "thin", with the exception
of ScrollbarThemeMac which uses NSSmallControlSize and
NSRegularControlSize to size and display the appropriate scrollbar.

ComputedStyle::HasCustomScrollbarStyle() returns true if a custom
style for the scrollbar has been set via ::-webkit-scrollbar and
the value of scrollbar-width has not been changed from the default.
This method replaces HasPseudoElementStyle(kPseudoIdScrollbar)
throughout the codebase so the standard properties for styling
scrollbars take precedence over the non-standard method.

WPT tests are included.

Bug: 891944
Change-Id: I9f77d46fc1b05a8d05bb4eeffeef4f8e52f8031b

Reviewed-on: https://chromium-review.googlesource.com/2624189
WPT-Export-Revision: f6dfdd84ac22ad1428719fb98af0adb024c2ab67

Component: web-platform-tests → Layout: Scrolling and Overflow
Product: Testing → Core
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.