Closed Bug 1579767 Opened 5 years ago Closed 5 years ago

Search one-offs container has incorrect accessibility role

Categories

(Firefox :: Address Bar, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 71
Tracking Status
firefox69 --- unaffected
firefox70 + verified
firefox71 --- verified

People

(Reporter: Jamie, Assigned: Jamie)

References

(Regression)

Details

(Keywords: access, regression)

Attachments

(1 file)

STR (with the NVDA screen reader):

  1. Press alt+d to focus the address bar.
  2. Type: test
  3. Press up arrow. You may need to press it twice if bug 1567384 comment 16 isn't fixed yet.
    • Expected: NVDA should say "This time, search with: grouping"
    • Actual: NVDA says "This time, search with: box"

This was regressed in bug 1561894. Previously, this was a <description> with role="group". In bug 1561894, the structure was changed and the role="group" was removed.

Assignee: nobody → jteh
Pushed by mzehe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bf56deedc6b7 Reinstate the accessibility group role for the search one-offs container. r=dao
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71

please ask for uplift when possible.

[Tracking Requested - why for this release]: accessibility regression

Comment on attachment 9091341 [details]
Bug 1579767: Reinstate the accessibility group role for the search one-offs container.

Beta/Release Uplift Approval Request

  • User impact if declined: Minor incorrect information reported for accessibility clients such as screen readers.
  • 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): Trivial patch which just adds a markup attribute that only impacts accessibility.
  • String changes made/needed: None.
Attachment #9091341 - Flags: approval-mozilla-beta?

Comment on attachment 9091341 [details]
Bug 1579767: Reinstate the accessibility group role for the search one-offs container.

Accessibility fix for new regression in 70, let's uplift for beta 7.

Attachment #9091341 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flags: qe-verify+
QA Whiteboard: [qa-triaged]

I have managed to reproduce this issue using Firefox 70.0a1 (BuildId:20190801214227) on Windows 10 64bit (with NVDA enabled).

This issue is verified fixed using Firefox 71.0a1 (BuildId:20190916155843) and Firefox 70.0b7 (BuildId:20190916074538) on Windows 10 64bit with NVDA.

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

Attachment

General

Created:
Updated:
Size: