With dark LW-themes, not the built-in dark theme, the toolbarbuttons don't highlight with a white background but a darker, which isn't good visible. This applies on Linux and Windows, Mac isn't affected.
With a dark LW-theme, not the built-in dark theme, applied hover over the toolbarbuttons and check how they are highlighted. On Linux I also removed no more needed variables I missed to remove. This all is a regression from the WX-theme work.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #9018860 - Flags: review?(jorgk)
The same patch applying on beta, when you still want to build a 63 beta.
The ESR patch.
Attachment #9018862 - Flags: approval-comm-esr60?
Oops, sorry forgot the reviewer in the commit message. Please add it when landing.
Comment on attachment 9018860 [details] [diff] [review] 1500742-button-LW-themes.patch Works OK, tested with "Dark Fox" and "Suave".
Attachment #9018860 - Flags: review?(jorgk) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/comm-central/rev/1c2a3007c1f5 Fix the toolbarbutton colors on dark LW-themes. r=jorgk DONTBUILD
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
The LW theme preview is distorted :-( - That's an M-C bug?
Comment on attachment 9018861 [details] [diff] [review] 1500742-button-LW-themes-beta.patch Thanks, but not doing TB 63 beta 2.
Attachment #9018862 - Flags: approval-comm-esr60? → approval-comm-esr60+
You need to log in before you can comment on or make changes to this bug.