Closed Bug 1707872 Opened 4 years ago Closed 3 years ago

Sidebar does not follow Firefox theme

Categories

(Core :: Widget: Gtk, defect)

Firefox 90
defect

Tracking

()

RESOLVED FIXED
Tracking Status
firefox91 --- fixed

People

(Reporter: jx99zz, Assigned: emilio)

References

Details

Attachments

(4 files, 1 obsolete file)

Attached image Screenshot (deleted) —

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.

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.

Component: Untriaged → Widget: Gtk
Product: Firefox → Core
Status: UNCONFIRMED → NEW
Component: Widget: Gtk → Theme
Ever confirmed: true
Product: Core → Firefox
Depends on: 1707895

Ah, I think we can fix this in the GTK code much better.

Assignee: nobody → emilio
Component: Theme → Widget: Gtk
No longer depends on: 1707895
Product: Firefox → Core
Depends on: 1707957

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.

Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/03767acc6135 Extract per-theme data in nsLookAndFeel into its own struct. r=stransky

Err, that should've landed in bug 1707957, oh well.

Keywords: leave-open
Depends on: 1708163
Flags: needinfo?(emilio)

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.

Attachment #9218708 - Attachment is obsolete: true
Blocks: 1714126

No behavior change intended.

Also no behavior change intended.

Depends on D117226

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

Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/35f93c68fc06 Factor out some code in Gtk's LookAndFeel. r=stransky
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0e6c67113ca7 Make GTK theme follow the firefox theme globally. r=stransky
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/780b3e476e83 Make LookAndFeel log its own module. r=stransky
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Regressions: 1716285
Regressions: 1716506
Regressions: 1725917
Regressions: 1782623
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: