linuxmint / mint-19.1-beta

BETA Bug Squah Rush
3 stars 3 forks source link

Firefox Mint-Y-Dark and dark themes in general Bug + Fix #30

Closed smurphos closed 5 years ago

smurphos commented 5 years ago

Describe the bug Longstanding readability issues with firefox content widget display in using Mint-Y-Dark and any other dark theme

To Reproduce Steps to reproduce the behavior: Set Mint-Y-Dark as GTK / Controls theme - experience text readability issues in various Firefox content widgets - particularly entries.

Expected behavior No readability issues.

Frequency Does the issue happen:

Edition (Desktop environment): In which edition is this happening?

Live or post-install: Indicate if this is happening after the installation, or during the live session:

The Fix

Ship the Mint Firefox build with an override in /usr/lib/firefox/distribution/distribution.ini to create and set this key in about:config

widget.content.gtk-theme-override with a value of Mint-Y

JosephMcc commented 5 years ago

Personally I'm not sure this is a great idea. It would be unclear to most users why Firefox isn't adapting to their theme changes.

smurphos commented 5 years ago

It doesn't effect the application of the GTK theme to the firefox header bar / menus etc only to the elements of website content that firefox partially themes.

JosephMcc commented 5 years ago

Hmm... I didn't realize that. I'd have to play with it a bit. It would raise bug reports from people who choose say the blue version and get stuck with green highlights, correct?

smurphos commented 5 years ago

Possibly or you could just go with Adwaita and be done with it. I don't really understand why users would want website content to be themed by the GTK theme anyway. Chromium based browsers don't do it.

gm10 commented 5 years ago

Personally I have it set to

user_pref("widget.content.gtk-theme-override", "#");

That (or any other invalid but not empty name) makes it fall back to its default, un-themed controls (at least on MATE). Seems preferable to me to using a specific theme.

Or just wait for Mozilla to fix this, the bug report on this has only been open for 18 years now. :100:

smurphos commented 5 years ago

user_pref("widget.content.gtk-theme-override", "#"); works well... screenshot-screen-2018-12-08-164936

NikoKrause commented 5 years ago

So, do I get it right, that if you choose a Dark variant for Firefox, e.g. Adwaita-Dark or Mint-Y-Dark-Red, you want Firefox to use/gtk-theme-override to Adwaita or Mint-Y-Red.

At least in Cinnamon it's possible to do this on theme change. We would just need to create or change the file /home/USER/.mozilla/firefox/***.default/user.js with the content:

user_pref("widget.content.gtk-theme-override", "My-Theme-without-Darkl");
gm10 commented 5 years ago

@NikoKrause that doesn't seem a reliable solution. Not all dark themes have a convenient -Dark suffix. The global override seems far more reliable, and also you're not taking the decision away from the user that way by simply modifying their local profile.

smurphos commented 5 years ago

Agreed with @gm10 - I think users would be happiest to have their web content delivered in a consistent readable fashion as intended by the web site designer without interference from their choice of system theme.

clefebvre commented 5 years ago

Fixed in ubuntu-system-adjustments 2018.12.12.