Closed Bug 1851110 Opened 2 years ago Closed 2 years ago

The keyboard-selected option for the address bar is not visible with dark default theme

Categories

(Firefox :: Theme, defect)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
Tracking Status
firefox-esr102 --- unaffected
firefox-esr115 --- unaffected
firefox117 --- unaffected
firefox118 --- unaffected
firefox119 --- verified

People

(Reporter: atrif, Assigned: emilio)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file, 1 obsolete file)

Attached image system_theme.gif

Found in

  • 119.0a1 (2023-08-31)

Affected versions

  • 119.0a1 (2023-08-31)

Tested platforms

  • Affected platforms: macOS 12, Windows 10x64
  • Unaffected platforms: Ubuntu 22.1

Preconditions

  • system theme set to dark
  • Firefox set to system theme

Steps to reproduce

  1. Open Firefox type something in the URL bar.
  2. Use the keyboard key to navigate the options.

Expected result

  • The selected option is visible.

Actual result

  • The selected option is not visible.

Regression range

Additional notes

  • Attached a screen recording.
  • The mouse hover effect is visible.

:emilio, since you are the author of the regressor, bug 1843044, could you take a look?

For more information, please visit BugBot documentation.

Flags: needinfo?(emilio)
Assignee: nobody → emilio

This used to be achieved via the dark theme, but I overlooked that there
was no browser-custom-colors equivalent.

On the light theme we were relying on Field / FieldText being black /
white, so make that explicit too:

InspectorUtils.colorToRGBA("Field", document) // Object { a: 1, b: 255, g: 255, r: 255 }
InspectorUtils.colorToRGBA("FieldText", document) // Object { a: 1, b: 0, g: 0, r: 0 }.

Flags: needinfo?(emilio)
Attachment #9351167 - Attachment is obsolete: true

Fixed by backout, will re-land with this fixed.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Flags: qe-verify+

Reproduced this on Win 10 and macOS 11 on a nightly build from 2023-08-31.
Verified as fixed on Firefox 119.0b4 (20231002091755) on the above mentioned OSes.

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

Attachment

General

Created:
Updated:
Size: