VOOZH about

URL: https://bugzilla.mozilla.org/show_bug.cgi?id=1950995

⇱ 1950995 - Remove browser.display.use_system_colors pref


Closed Bug 1950995 Opened 1 year ago Closed 2 months ago

Remove browser.display.use_system_colors pref

Remove browser.display.use_system_colors pref
Firefox
Disability Access
unspecified
Unspecified
Unspecified
task
Points:
---
RESOLVED FIXED
RESOLVED
FIXED
151 Branch
Iteration:
---
a11y-review
Accessibility Severity
Performance Impact
Webcompat Priority
Webcompat Score
Tracking Status
firefox151 --- fixed
Tracking Status
relnote-firefox
firefox-esr115
firefox-esr140
firefox-esr153
firefox151
firefox152
firefox153
firefox154
---
QA Whiteboard:
[qa-triage-done-c152/b151]
Has STR:
---
Change Request:
---
Bug Flags:
Signature:
None
This bug is publicly visible.

 

After bug 1898096, use_system_colors will not be exposed in UI and won't have any affect. It will need to be removed along with its telemetry probes.

I'm separating this task from bug 1898096 in case the changes there need to be reverted so we won't have to re-introduce a pref and telemetry probe.

I don't believe that this should occur. Currently, this is the sole feasible way for me to override web content with my OS's colouration. Firefox provides no alternative except manually injecting :root { color-scheme: light dark } into every webpage I visit, etcetera.

This isn't merely an accessibility preference, as discuss.kde.org/t/29444/15 corroborates: it also provides theming. Unless id=1888225 is reconsidered, all about: pages are now OS-colouration inadherent.

Comment 2

1 year ago

It is very uncommon in on the web today to encounter unstyled content.

Surely web pages is not just text files. However over half of the web pages I visit do not offer light and dark themes at the same time.

Now with the setting and **any **dark DE, CONTRAST SETTINGS > Automatic (Use system settings) will expose the user to bright webpages (not just white) and their eyes.
This does not go well aesthetically with dark window decorations either so it looks broken.
I'm speaking for Linux, could not test anything else.

The ability to change the default colors without high contrast mode does not offer a good experience.

It provided a perfect experience for me. Of course, banners of web pages did not often change colors but the rest did very well and felt like a home experience.
One was able to inject the DE's accent and theming colors into the web layout. Now the whole thing just turns grayish.
One was able to inject the DE's accent and theming colors which adapted with the wallpaper you use. Perfect consistent theming while browsing. Perfect integration into KDE Plasma. _Gone _ with the wind!

There are three color palettes to choose from in light of the preference
above they are:

  1. WIDGET_COLORS: The OS's configured colors used by its theme.

This is not true for KDE Plasma 5 or 6! No colors do get into Firefox web backgroud or fonts anymore!

CONTRAST SETTINGS > Automatic (Use system settings) does not include KDE Plasma 5.27.12 or KDE Plasma 6.3.5 colors into web pages.
Neither does it consistently provide a dark theming without using third party add-ons like Dark Reader (which lets webpages blink up white before theming them dark for a fraction of a second as well).

PLEASE bring back the old setting:
Colors > Use System Colors | Always

I don't see any reason this was removed.
Bring it back into - Custom or:
Make it at least available in about:config as a fallback for users who don't want the Contrast Control setting if they use their browser theming (consistent with wallpaper and all web content!)

Cheers a disappointed KDE Plasma costumizing user!

Comment 3

1 year ago

(^ quotation incorrect in above post, please remove.)

It is very uncommon in on the web today to encounter unstyled content.

Surely web pages is not just text files. However over half of the web pages I visit do not offer light and dark themes at the same time.

Now with the setting and **any **dark DE, CONTRAST SETTINGS > Automatic (Use system settings) will expose the user to bright webpages (not just white) and their eyes.
This does not go well aesthetically with dark window decorations either so it looks broken.
I'm speaking for Linux, could not test anything else.

The ability to change the default colors without high contrast mode does not offer a good experience.
It provided a perfect experience for me. Of course, banners of web pages did not often change colors but the rest did very well and felt like a home experience.
One was able to inject the DE's accent and theming colors into the web layout. Now the whole thing just turns grayish.
One was able to inject the DE's accent and theming colors which adapted with the wallpaper you use. Perfect consistent theming while browsing. Perfect integration into KDE Plasma. _Gone _ with the wind!

There are three color palettes to choose from in light of the preference
above they are:

  1. WIDGET_COLORS: The OS's configured colors used by its theme.

This is not true for KDE Plasma 5 or 6! No colors do get into Firefox web backgroud or fonts anymore!

CONTRAST SETTINGS > Automatic (Use system settings) does not include KDE Plasma 5.27.12 or KDE Plasma 6.3.5 colors into web pages.
Neither does it consistently provide a dark theming without using third party add-ons like Dark Reader (which lets webpages blink up white before theming them dark for a fraction of a second as well).

PLEASE bring back the old setting:
Colors > Use System Colors | Always

I don't see any reason this was removed.
Bring it back into - Custom or:
Make it at least available in about:config as a fallback for users who don't want the Contrast Control setting if they use their browser theming (consistent with wallpaper and all web content!)

Cheers a disappointed KDE Plasma costumizing user!

Comment 4

1 year ago

browser.display.use_system_colors
is still in about:config
But is not working the way it did before the Contrast Control UI settings were introduced!

WIDGET_COLORS: The OS's configured colors used by its theme.

This is not true for KDE Plasma 5 or 6! No colors do get into Firefox web backgroud or fonts anymore! "Contrast Settings" > "Automatic" (Use system settings) does not include KDE Plasma 5.27.12 or KDE Plasma 6.3.5 colors into web pages.

On that note, although I've no introspection into the browser source (I don't understand how to utilise Mercurial), the previous implementation adhered to the QPalette that KColorScheme exposes. Now, although it continues to adhere to Qt::ColorScheme (at least, as exposed by org.freedesktop.appearance's color-scheme), it always sets brand colours, instead of what the QPalette defines.

I really think that at least either this or id=1888225 need to be reconsidered, since although I can understand wanting to remove a non-CSS-based CSS override, I do not understand taking away the sole workaround for having the entire browser themed like the DE.

Comment hidden (off-topic)

I'm a new contributor, and I'll be submitting a patch for this soon.

Assignee: nobody → dev
Attachment #9557138 - Attachment description: WIP: Bug 1950995 - Remove browser.display.use_system_colors pref. → Bug 1950995 - Remove browser.display.use_system_colors pref. r?eeejay!
Status: NEW → ASSIGNED

Comment 10

2 months ago
Pushed by ealvarez@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/96ca091b925b https://hg.mozilla.org/integration/autoland/rev/34343931ea62 Remove browser.display.use_system_colors pref. r=eeejay,extension-reviewers,chutten,robwu

Comment 11

2 months ago
bugherder
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 151 Branch
QA Whiteboard: [qa-triage-done-c152/b151]
You need to log in before you can comment on or make changes to this bug.