in Dark theme, text contrast ratio is too low for "Search with Google or enter address"
Categories
(Firefox :: Theme, defect, P3)
Tracking
()
People
(Reporter: billdillensrevenge, Assigned: dao)
References
Details
(Keywords: access)
Attachments
(3 files)
20.26 KB,
image/png
|
Details | |
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
26.22 KB,
image/png
|
Details |
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:69.0) Gecko/20100101 Firefox/69.0
Steps to reproduce:
in Dark theme, the text contrast ratio of "Search with Google or enter address" in the URL bar is too low, please see attached screenshot comparing Firefox light mode to dark mode (light mode has good contrast ratio)
Comment 1•4 years ago
|
||
Bugbug thinks this bug should belong to this component, but please revert this change in case of error.
Theme but also Accessibility (I have good vision and an excellent calibrated display and I still can't easily or quickly make it out)
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 3•4 years ago
|
||
Assignee | ||
Updated•4 years ago
|
Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/447c2f04371a Remove obsolete urlbar and searchbar placeholder text styling. r=ntim
Is this bug a duplicate or just very similar? Please take a look https://bugzilla.mozilla.org/show_bug.cgi?id=1567714
Assignee | ||
Comment 6•4 years ago
•
|
||
(In reply to Will from comment #5)
Is this bug a duplicate or just very similar? Please take a look https://bugzilla.mozilla.org/show_bug.cgi?id=1567714
It's just similar.
Comment 7•4 years ago
|
||
bugherder |
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 8•4 years ago
|
||
Comment on attachment 9086687 [details]
Bug 1574736 - Remove obsolete urlbar and searchbar placeholder text styling. r=ntim
Beta/Release Uplift Approval Request
- User impact if declined: urlbar and searchbar placeholder text has low contrast on Windows when using a dark webextension themes (including our own Dark theme)
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: see comment 0
- List of other uplifts needed: -
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Removing an obsolete CSS rule that should have been removed a long time ago
- String changes made/needed: -
Comment 9•4 years ago
|
||
Comment on attachment 9086687 [details]
Bug 1574736 - Remove obsolete urlbar and searchbar placeholder text styling. r=ntim
CSS-only fix to improve dark mode styling. Approved for 69.0b16.
Reporter | ||
Comment 10•4 years ago
|
||
This issue also applies to the separate search bar (Options > Search > Add search bar in toolbar), does this fix it for both or does the search bar also need to be fixed? (attaching screenshot in a moment)
Reporter | ||
Comment 11•4 years ago
|
||
Comment 12•4 years ago
|
||
bugherderuplift |
Comment 13•4 years ago
|
||
Confirming this issue as verified fixed on 70.0a1 (buildID: 20190821215524) and 69.0b16 (buildID: 20190821213409). Verified using Windows 10x64 , macOS 10.14.6 and Ubuntu 18.04x64.
The "Search with Google or enter address" and the separate search bar (Options > Search > Add search bar in toolbar) are now both visible while browser is in Dark Mode.
Assignee | ||
Updated•3 years ago
|
Description
•