Closed
Bug 1501586
Opened 6 years ago
Closed 6 years ago
Clean up lwt-*-brighttext attributes properly when switching from a dark theme to a light theme
Categories
(WebExtensions :: Themes, defect)
WebExtensions
Themes
Tracking
(firefox65 verified)
VERIFIED
FIXED
mozilla65
Tracking | Status | |
---|---|---|
firefox65 | --- | verified |
People
(Reporter: ntim, Assigned: ntim)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
I've tested only on MacOS so far.
STR:
- Apply the patch from bug 1485599
- Install https://addons.mozilla.org/en-US/firefox/addon/night_owl/ on the build
- Open the bookmarks sidebar
- Click the "owl" toolbar icon from the extension
AR:
- The scrollbar doesn't change colors
ER:
- The scrollbar should have changed colors
Assignee | ||
Updated•6 years ago
|
Summary: Scrollbars should be re-rendered when their scrollbar-color properties are changed → Scrollbars should be re-rendered when their scrollbar-color property is changed
Assignee | ||
Comment 1•6 years ago
|
||
(oh, I forgot one step in the STR: you need to resize the window so scrollbars appear in the sidebar)
Comment 2•6 years ago
|
||
It'd be extra-great if you could attach an HTML-only test-case :)
This is supposed to work at least:
https://searchfox.org/mozilla-central/rev/a7f4d3ba4fbfe3efbde832869f1d672fce7122f6/layout/style/nsStyleStruct.cpp#4631
Xidorn, any thought?
Flags: needinfo?(xidorn+moz)
Assignee | ||
Comment 3•6 years ago
|
||
I can't seem to make an HTML-only test case, so it's probably a very specific edge case.
Comment 4•6 years ago
|
||
I have to sleep now... but some random thought: this may be related to how XUL handles scrollbars which, IIRC, is a different code path and component than what is used in HTML.
Comment 5•6 years ago
|
||
This is not a bug of the rendering engine. It is a bug of the frontend code somewhere.
When dark theme is used, "lwt-sidebar-brighttext" attribute is added to the page.sidebar-panel element, and it matches .sidebar-panel[lwt-sidebar-brighttext] which changes the scrollbar-color. When switching back to light theme, this attribute is *not* removed, and thus scrollbar-color is still from .sidebar-panel[lwt-sidebar-brighttext], which is light.
If I close the sidebar in light theme and reopen it, there is no "lwt-sidebar-brighttext" attribute on that element.
So this is neither a style system bug nor a rendering bug. It is most likely a bug in frontend code somewhere which doesn't setup attributes correctly.
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(xidorn+moz)
Resolution: --- → INVALID
Assignee | ||
Comment 6•6 years ago
|
||
Thanks for looking into it! Reopening this so I can fix this in the WebExtension side.
Status: RESOLVED → REOPENED
Component: CSS Parsing and Computation → Themes
Product: Core → WebExtensions
Resolution: INVALID → ---
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → ntim.bugs
Status: REOPENED → ASSIGNED
Summary: Scrollbars should be re-rendered when their scrollbar-color property is changed → Clean up lwt-*-brighttext attributes properly when switching from a dark theme to a light theme
Assignee | ||
Comment 7•6 years ago
|
||
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a2b59c32f04d
Clean up lwt-*-brighttext attributes properly when switching from a dark theme to a light theme. r=jaws
Comment 9•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago → 6 years ago
status-firefox65:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Comment 10•6 years ago
|
||
Verified using the latest Nightly (65.0a1) on Windows 10 x64 and MacOs according to the steps provided in the description, the scrollbar along with all of the sidebar seems to change colors according to the Night Owl scheme, the issue seems to be resolved.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•