[Non-Proton] Search bar's keyboard highlight is barely visible in Firefox Dark theme
Categories
(Firefox :: Theme, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr78 | --- | unaffected |
firefox87 | --- | unaffected |
firefox88 | --- | unaffected |
firefox89 | --- | wontfix |
People
(Reporter: Fanolian+BMO, Unassigned)
References
(Depends on 1 open bug, Regression)
Details
(Keywords: nightly-community, regression, reproducible, Whiteboard: [proton-address-bar] [priority:2b] [a11y])
Attachments
(1 file)
42.01 KB,
image/png
|
Details |
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:89.0) Gecko/20100101 Firefox/89.0
Build ID: 20210331215444
Settings
Win10 OS theme: Light or Dark
Nightly theme: Dark
No Proton related stuff is enabled in Nightly.
Steps to reproduce
- Add search bar to UI.
- Type something in the bar.
- Use keyboard to highlight a search suggestion/history.
Actual result
The highlight colour is barely visible.
In the attached screenshot, in both before and after the 2nd result is highlighted by keyboard navigation while 4th by mouse hover.
Notes
- Nightly's Default theme is unaffected even if OS is set to dark theme.
- URL bar is not affected.
The bug should be first introduced by bug 1692046 when Dark and Proton Dark were separated, and went through some more iterations. Now Proton Dark replaces Dark and, iiuc, rides the train with bug 1701256, this bug will affect non-Proton users at Release 89.
Comment 1•4 years ago
|
||
:Fanolian+BMO, since this bug is a regression, could you fill (if possible) the regressed_by field?
For more information, please visit auto_nag documentation.
Updated•4 years ago
|
Updated•4 years ago
|
I think changing these vars should fix it:
https://searchfox.org/mozilla-central/rev/54f37fc1ac0f98b590af51e01ce82bb74179bf63/browser/themes/shared/urlbarView.inc.css#15-18
maybe to match vars like --lwt-toolbar-field-focus
(which is what the urlbar uses) etc.
Comment 3•4 years ago
|
||
Set release status flags based on info from the regressing bug 1701256
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Comment 4•4 years ago
|
||
Considering this works correctly in Proton, and Proton is being release in 89, I think this is a wontfix.
Updated•4 years ago
|
Description
•