Closed
Bug 1227636
Opened 9 years ago
Closed 9 years ago
Fennec with C++ APZ has no scrollbars
Categories
(Firefox for Android Graveyard :: Toolbar, defect)
Tracking
(firefox45 fixed)
RESOLVED
FIXED
Firefox 45
Tracking | Status | |
---|---|---|
firefox45 | --- | fixed |
People
(Reporter: kats, Assigned: kats)
References
Details
Attachments
(1 file, 1 obsolete file)
1.47 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
In bug 1223928 I enabled the Gecko scrollbars for Fennec with C++ APZ, but at around the same time some theming changes landed in bug 1223526 which made it not work again. In particular there appear to be some styles that we used to be pulling in from toolkit/themes/windows/global/xulscrollbars.css that we are no longer doing, and that is causing the scrollbars to not show up. I was under the impression that all the relevant styles were already in mobile/android/themes/core/content.css but that's not the case, apparently. The attached WIP does fix the scrollbars by copying the windows scrollbar styles back into the mobile theme, but it's probably overkill to take all those styles when we're probably just missing a couple. I suspect a better approach would be to take the scrollbar-related styles from content.css, merge them with the windows scrollbar styles, throw out anything that's not needed, and save the rest into a new override file in mobile/android/themes/core. However my knowledge of the theming/packaging of this stuff is almost nil so I don't know if that even makes sense.
Assignee | ||
Comment 1•9 years ago
|
||
Looks like it was just a few missing properties. Putting those into content.css fixes the issue.
Attachment #8691466 -
Attachment is obsolete: true
Attachment #8691485 -
Flags: review?(mark.finkle)
Comment 2•9 years ago
|
||
Comment on attachment 8691485 [details] [diff] [review] Patch Nice!
Attachment #8691485 -
Flags: review?(mark.finkle) → review+
Comment 4•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/85af1a62e15a
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 45
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•