Closed Bug 1619428 Opened 5 years ago Closed 5 years ago

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)

task
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla75
Tracking Status
firefox75 --- fixed

People

(Reporter: mstange, Assigned: emilio)

References

Details

Attachments

(1 file)

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.

Blocks: 1618202
No longer blocks: 1615105
Flags: needinfo?(emilio)
Assignee: nobody → emilio

Reuse the AddXULMinSize logic which already deals with all the widget stuff,
non-themed scrollbars, etc.

Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/be3b005bfc6d Make the overlay scrollbars check work in all platforms. r=mstange
Backout by shindli@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/095ba181a9dc Backed out changeset be3b005bfc6d for causing reftest failure in contain-size-select-elem-002-ref.html CLOSED TREE
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/76f13c8e10b7 Make the overlay scrollbars check work in all platforms. r=mstange
Flags: needinfo?(emilio)
Pushed by emilio@crisal.io: https://hg.mozilla.org/integration/autoland/rev/58d71374ca38 Annotate a test as passing that should've been annotated in the previous patch. CLOSED TREE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: