The non-native theme should advertise itself as supporting StyleAppearance::Scrollbar, and should set an appropriate minimum size for StyleAppearance::Scrollbar
Categories
(Core :: Widget, task)
Tracking
()
Tracking | Status | |
---|---|---|
firefox75 | --- | fixed |
People
(Reporter: mstange, Assigned: emilio)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
When overlay scrollbars are enabled (macOS and Android by default, other platforms if you set the ui.useOverlayScrollbars
pref), scrollbars shouldn't take up any layout space. The way this works on the platforms it works on is that nsScrollbarFrame::GetXULMargin
sets a negative margin on the scrollbar that is equal to its size. However, it only does that if theme->ThemeSupportsWidget(presContext, this, StyleAppearance::Scrollbar)
. It seems that Windows (?), Linux and the non-native theme don't return true in this call, so overlay scrollbars incorrectly take up space.
Reporter | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Reuse the AddXULMinSize logic which already deals with all the widget stuff,
non-themed scrollbars, etc.
Assignee | ||
Updated•5 years ago
|
Comment 3•5 years ago
|
||
Backed out changeset be3b005bfc6d (Bug 1619428) for causing reftest failure in contain-size-select-elem-002-ref.html CLOSED TREE
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=291488460&repo=autoland&lineNumber=5860
Assignee | ||
Updated•5 years ago
|
Comment 10•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/76f13c8e10b7
https://hg.mozilla.org/mozilla-central/rev/c26f12bfc75c
https://hg.mozilla.org/mozilla-central/rev/58d71374ca38
https://hg.mozilla.org/mozilla-central/rev/07c44cc38c7b
Description
•