Closed Bug 1708799 Opened 3 years ago Closed 3 years ago

Insufficient contrast on action text on hover in dark mode

Categories

(Firefox :: Address Bar, defect, P2)

defect
Points:
1

Tracking

()

VERIFIED FIXED
90 Branch
Iteration:
90.1 - Apr 19 - May 2
Tracking Status
firefox89 + verified
firefox90 --- verified

People

(Reporter: bugzilla, Assigned: bugzilla)

References

(Blocks 1 open bug)

Details

(Whiteboard: [proton-address-bar] [proton-uplift])

Attachments

(2 files)

UX noticed that there's not enough contrast between action text and hovered rows in the Urlbar in dark mode. Screenshot attached. They suggested using the title color for the action text on hover, like we do for keyboard-selected rows. The Figma spec has been updated to reflect this.

Whiteboard: [proton-address-bar]
Assignee: nobody → htwyford
Status: NEW → ASSIGNED
Iteration: --- → 90.1 - Apr 19 - May 2

[Tracking Requested - why for this release]:
Minor a11y issue; Proton regression; very simple CSS fix.

Pushed by htwyford@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5a084bd46f24 Set action text to title color on mouse hover. r=adw

Comment on attachment 9219630 [details]
Bug 1708799 - Set action text to title color on mouse hover. r?adw!

Beta/Release Uplift Approval Request

  • User impact if declined: Small a11y Proton regression in the address bar. Some text will have insufficient contrast.
  • 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: The bug fix will be easier to notice if you do these steps in dark mode.
  1. Go to https://bugzilla.mozilla.org/ and bookmark it.
  2. Right click the address bar and select "Add Bugzilla @ Mozilla"
  3. Close the tab.
  4. Start typing "bugzilla" until bugzilla.mozilla.org is autofilled and the "Search with Bugzilla @ Mozilla" tab-to-search result is displayed as the second result.
  5. Observe that the text "Search with Bugzilla @ Mozilla directly from the address bar" is grey
  6. Press the down arrow key to select the result. Observe that the text "Search with Bugzilla @ Mozilla directly from the address bar" is now the same colour as other text in the address bar (white in dark mode, black in light mode).
  7. Use the arrow keys to unselect the result. Now, hover it with the mouse. Observe that the text "Search with Bugzilla @ Mozilla directly from the address bar" is now the same colour as other text in the address bar.
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Two-line CSS change. Well understood, narrowly scoped.
  • String changes made/needed:
Attachment #9219630 - Flags: approval-mozilla-beta?
Flags: qe-verify+
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
Whiteboard: [proton-address-bar] → [proton-address-bar] [proton-uplift]
QA Whiteboard: [qa-triaged]

Comment on attachment 9219630 [details]
Bug 1708799 - Set action text to title color on mouse hover. r?adw!

Low risk, approved for 89 beta 8, thanks.

Attachment #9219630 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

I can confirm the figma spec is respected. Following the steps from comment 4 I verified using Fx 90.0a1(2021-05-05) and Fx 89.0b8 on Windows 10 x64, macOS 10.13.6 and Ubuntu 20.04 LTS.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: