Closed Bug 1735643 Opened 3 years ago Closed 3 years ago

Urlbar search mode indicator invisible in HCM when the urlbar is not focused

Categories

(Firefox :: Address Bar, defect, P1)

defect
Points:
1

Tracking

()

VERIFIED FIXED
95 Branch
Iteration:
95.2 - Oct 18 - Oct 31
Accessibility Severity s2
Tracking Status
firefox-esr78 --- unaffected
firefox-esr91 --- unaffected
firefox93 --- unaffected
firefox94 + verified
firefox95 --- verified

People

(Reporter: itiel_yn8, Assigned: bugzilla)

References

(Regression)

Details

(Keywords: access, regression, Whiteboard: [fidefe-theme][hcm-2021-h2])

Attachments

(2 files)

Attached image Screenshot (deleted) —

Tested on Windows with HCM White.

Reproduced on HCM when the address bar dropdown panel is opened and search mode chiclet is selected.
When the dropdown is dismissed with Escape key, the chiclet won't be visible. If the focus is moved in-page, the chiclet will become visible.

Mozregression returned Bug 1730237 - Split --urlbar-box-focus-bgcolor out from --urlbar-box-bgcolor. r=desktop-theme-reviewers,dao as regressor, Harry could you please take a look? Thank you!

Flags: needinfo?(htwyford)
Keywords: access, regression
Regressed by: 1730237
Whiteboard: [access-s2]
Has Regression Range: --- → yes

[Tracking Requested - why for this release]:
UI element is invisible in HCM. Low-risk, CSS-only fix is possible.

Assignee: nobody → htwyford
Severity: -- → S2
Status: NEW → ASSIGNED
Iteration: --- → 95.2 - Oct 18 - Oct 31
Points: --- → 1
Flags: needinfo?(htwyford)
Priority: -- → P1
Blocks: 1725467
Whiteboard: [access-s2] → [access-s2], [fidefe-theme]

This patch unifies the HCM colors used in the identity box, the search mode indicator, and the zoom indicator. It uses the HCM convention used in VS Code of using dotted and solid offset outlines to show hover and active states.

Morgan, the attached patch adds a dotted inset outline on an element to denote focus in HCM. The outline becomes solid on :hover:active. I've seen that done in VS Code and it was the best model I could think of here, given the element uses a vibrant background color. What do you think? Is there a better way to show hover/active?

Flags: needinfo?(mreschenberg)

(In reply to Harry Twyford [:harry] from comment #4)

Morgan, the attached patch adds a dotted inset outline on an element to denote focus in HCM. The outline becomes solid on :hover:active. I've seen that done in VS Code and it was the best model I could think of here, given the element uses a vibrant background color. What do you think? Is there a better way to show hover/active?

Thanks for patching this! I'm gonna give it a spin on Windows in a few different HCM themes, and then I'll follow up here with thoughts :)

Added comments on phab :)

Flags: needinfo?(mreschenberg)
Whiteboard: [access-s2], [fidefe-theme] → [access-s2] [fidefe-theme]
Pushed by htwyford@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/46edc3da8b6c Unify Urlbar box HCM behaviour. r=desktop-theme-reviewers,dao
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 95 Branch

Comment on attachment 9245928 [details]
Bug 1735643 - Unify Urlbar box HCM behaviour. r?#desktop-theme-reviewers!

Beta/Release Uplift Approval Request

  • User impact if declined: The search mode indicator in the Urlbar will be invisible in HCM.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): CSS-only, well-understood.
  • String changes made/needed:
Attachment #9245928 - Flags: approval-mozilla-beta?
Blocks: 1737054

Comment on attachment 9245928 [details]
Bug 1735643 - Unify Urlbar box HCM behaviour. r?#desktop-theme-reviewers!

Approved for 94.0b9.

Attachment #9245928 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Whiteboard: [access-s2] [fidefe-theme] → [access-s2] [fidefe-theme][hcm-2021-h2]

I've managed to reproduce the issue using Fx95.0a1.
The issue is verified fixed using Fx95.0b5 and latest Fx96.0a1 on Windows 10, Ubuntu 20.04 and macOS 11. The one-off ciclet is correctly visible with high contrast mode enabled.

Status: RESOLVED → VERIFIED
Accessibility Severity: --- → s2
Whiteboard: [access-s2] [fidefe-theme][hcm-2021-h2] → [fidefe-theme][hcm-2021-h2]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: