Match more browser components to the browser theme
Categories
(Firefox :: Theme, enhancement, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox89 | --- | verified |
People
(Reporter: mstriemer, Assigned: mstriemer)
References
(Blocks 1 open bug)
Details
(Whiteboard: [proton-infobars])
Attachments
(10 files, 2 obsolete files)
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details |
The infobars have a light/dark colour scheme, but this follows the OS-level preference, not the theme's. UX would prefer that we match the theme's preference (this is brighttext
in the code) rather than the OS.
STR
- Set your theme to light or dark, opposite your OS preference
- Trigger an infobar
- Go to https://permission.site/
- Click "Protocol Handler"
Expected results: The infobar colours match your theme's dark/light
Actual results: The infobar is light on dark or dark on light, matching the OS
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Comment 1•4 years ago
|
||
I separated out the prefers-color-scheme: dark
in common.inc.css so that we can a version with that and without that and use the version without that for infobars. Doing this eliminates the OS theme changes affect on the infobar. Unfortunately, it's still not perfect. The button styling seems to be off.
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
This also affects proton modals and infobars and whatever else
includes common.css in its shadow DOM in browser chrome.
Comment 3•4 years ago
|
||
Promoted to P1 after discussions in Slack surrounding MR1 triage.
Updated•4 years ago
|
Assignee | ||
Comment 4•4 years ago
|
||
Depends on D111486
Assignee | ||
Comment 5•4 years ago
|
||
Depends on D111970
Assignee | ||
Comment 6•4 years ago
|
||
Depends on D111971
Assignee | ||
Comment 7•4 years ago
|
||
Depends on D111972
Assignee | ||
Comment 8•4 years ago
|
||
Depends on D111973
Assignee | ||
Comment 9•4 years ago
|
||
Depends on D111974
Assignee | ||
Comment 10•4 years ago
|
||
Depends on D111975
Assignee | ||
Comment 11•4 years ago
|
||
Depends on D111976
Assignee | ||
Comment 12•4 years ago
|
||
Depends on D111977
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Comment 13•4 years ago
|
||
Depends on D111486
Comment 14•4 years ago
|
||
Comment 15•4 years ago
|
||
Backed out for failures on browser_toolbar_prefers_color_scheme.js
backout: https://hg.mozilla.org/integration/autoland/rev/48a99646f183abc34e8eeb695fcede5bcc9463d1
tier 1 failure seen on the next push: https://treeherder.mozilla.org/jobs?repo=autoland&selectedTaskRun=EqOo_4CkQduiGAkQZyvnsg.0&revision=f27f2a126a3a95f160ded1dc2c5802e44e964f52&searchStr=windows%2C10%2Cx64%2Copt%2Cmochitests%2Ctest-windows10-64%2Fopt-mochitest-browser-chrome-e10s%2Cbc7
failure log: https://treeherder.mozilla.org/logviewer?job_id=336694077&repo=autoland&lineNumber=3100
[task 2021-04-15T19:01:38.396Z] 19:01:38 INFO - TEST-PASS | browser/components/extensions/test/browser/browser_toolbar_prefers_color_scheme.js | OS dark mode unchanged -
[task 2021-04-15T19:01:38.396Z] 19:01:38 INFO - Buffered messages finished
[task 2021-04-15T19:01:38.396Z] 19:01:38 INFO - TEST-UNEXPECTED-FAIL | browser/components/extensions/test/browser/browser_toolbar_prefers_color_scheme.js | toolbar query is light mode -
[task 2021-04-15T19:01:38.396Z] 19:01:38 INFO - Stack trace:
[task 2021-04-15T19:01:38.396Z] 19:01:38 INFO - chrome://mochikit/content/browser-test.js:test_ok:1334
[task 2021-04-15T19:01:38.396Z] 19:01:38 INFO - chrome://mochitests/content/browser/browser/components/extensions/test/browser/browser_toolbar_prefers_color_scheme.js:testThemeDeterminesToolbarQuery:47
[task 2021-04-15T19:01:38.396Z] 19:01:38 INFO - Leaving test bound testThemeDeterminesToolbarQuery
[task 2021-04-15T19:01:38.396Z] 19:01:38 INFO - GECKO(3437) | MEMORY STAT | vsize 3114MB | residentFast 444MB | heapAllocated 200MB
[task 2021-04-15T19:01:38.396Z] 19:01:38 INFO - TEST-OK | browser/components/extensions/test/browser/browser_toolbar_prefers_color_scheme.js | took 418ms
Updated•4 years ago
|
Comment 16•4 years ago
|
||
Comment 17•4 years ago
|
||
Backed out changeset 1c9943dacbdb (Bug 1701691) for causing failures in browser_toolbar_prefers_color_scheme.js CLOSED TREE
The job didn't run on the original push because of some build bustages from another bug.
Log: https://treeherder.mozilla.org/logviewer?job_id=336747500&repo=autoland&lineNumber=30454
Backout: https://hg.mozilla.org/integration/autoland/rev/163d8a588c39ed388181a2b202c5235f95167adf
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Comment 19•4 years ago
|
||
Comment 20•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/7e864fb91533
https://hg.mozilla.org/mozilla-central/rev/86d38280fedc
https://hg.mozilla.org/mozilla-central/rev/e05824608cb9
https://hg.mozilla.org/mozilla-central/rev/4c299a07326b
https://hg.mozilla.org/mozilla-central/rev/07d036bff2a2
https://hg.mozilla.org/mozilla-central/rev/fa6845ddd370
https://hg.mozilla.org/mozilla-central/rev/69ffe4edadc1
https://hg.mozilla.org/mozilla-central/rev/054f9a3d20b1
https://hg.mozilla.org/mozilla-central/rev/5a498c95f727
Comment 22•4 years ago
|
||
Verified the fix on MacOS 11, Windows 10 and Ubuntu 20.04 using Firefox Nightly 90.0a1 (20210419221626)
Assignee | ||
Comment 23•4 years ago
|
||
Updating the description, since this updated our common styling. This updates the in-content pages (about:preferences, about:addons, etc that use our common styling), infobars, alerts, pdfjs, print modal, and the webrtc indicator (maybe not a definitive list). In general, if it's Firefox UI and the theme is "dark" then that Firefox UI should now be dark as well, rather than following the OS.
Updated•4 years ago
|
Updated•2 years ago
|
Description
•