Sidebar does not follow Firefox theme
Categories
(Core :: Widget: Gtk, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox91 | --- | fixed |
People
(Reporter: jx99zz, Assigned: emilio)
References
Details
Attachments
(4 files, 1 obsolete file)
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:87.0) Gecko/20100101 Firefox/87.0
Steps to reproduce:
Opened the bookmarks/history sidebar.
Actual results:
The sidebar still follows the Operating System's theme, not the current Firefox theme. This issue is most visible when using a dark Operating System theme with a light Firefox theme (or vice-versa).
Expected results:
The sidebar should be styled to follow Firefox's current theme. In older versions of Firefox, this wasn't a big deal, since much of the browser depended on the Operating System's theme. With the new Proton theme most of the browser is now manually styled, so it makes more sense to do so to the sidebar as well.
Comment 1•4 years ago
|
||
The Bugbug bot thinks this bug should belong to the 'Core::Widget: Gtk' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
Ah, I think we can fix this in the GTK code much better.
Assignee | ||
Comment 3•4 years ago
|
||
We're going to change the setup a bit to support both light and dark
system colors in the chrome process, like mac does. This is just
refactoring though.
Remove some unneeded IsContentProcess() check since it can't happen now
that we always enable RemoteLookAndFeel.
Assignee | ||
Comment 5•4 years ago
|
||
Err, that should've landed in bug 1707957, oh well.
Comment 6•4 years ago
|
||
bugherder |
Comment 7•4 years ago
|
||
Backed out for causing valgrind failures.
Backout link: https://hg.mozilla.org/integration/autoland/rev/3b6178fd74b18ca31856c11a688caea992b1433c
Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&revision=0fb0e3d056f8c73ac19a2c81f32e20935db0d19e&searchStr=valgrind&selectedTaskRun=LryxK1N5QLmdrY8o0Dd8pQ.0
Failure log: https://treeherder.mozilla.org/logviewer?job_id=337971274&repo=autoland&lineNumber=63906
Comment 8•4 years ago
|
||
Backout merged: https://hg.mozilla.org/mozilla-central/rev/3b6178fd74b1
Assignee | ||
Updated•4 years ago
|
Comment 9•4 years ago
|
||
Comment on attachment 9218708 [details]
Bug 1707872 - Extract per-theme data in nsLookAndFeel into its own struct. r=stransky
Revision D113542 was moved to bug 1708163. Setting attachment 9218708 [details] to obsolete.
Assignee | ||
Comment 10•3 years ago
|
||
No behavior change intended.
Assignee | ||
Comment 11•3 years ago
|
||
Also no behavior change intended.
Depends on D117226
Assignee | ||
Comment 12•3 years ago
|
||
This matches what macOS is doing in bug 1715145, and improves the
situation in the cases the user chooses a light Firefox theme, but a
dark GTK theme, or vice versa.
The nice thing of doing it globally is that we don't need to teach the
GTK code to paint with different color schemes in different windows,
which would be massively annoying.
This is expected to have an slight performance cost on startup for
light-theme users (as we need to read the dark gtk theme colors), but
it's hopefully ok.
Depends on D117227
Comment 13•3 years ago
|
||
Comment 14•3 years ago
|
||
Comment 15•3 years ago
|
||
Comment 16•3 years ago
|
||
bugherder |
Assignee | ||
Updated•3 years ago
|
Updated•3 years ago
|
Updated•3 years ago
|
Description
•