Closed Bug 1572641 Opened 5 months ago Closed 5 months ago

More accessibility fix-ups for screen readers

Categories

(Firefox :: Protections UI, defect, P1)

Desktop
All
defect

Tracking

()

RESOLVED FIXED
Firefox 70
Tracking Status
firefox70 --- fixed

People

(Reporter: MarcoZ, Assigned: MarcoZ)

References

(Regressed 1 open bug)

Details

(Keywords: access, Whiteboard: [privacy-panel][skyline])

Attachments

(4 files)

Very recent landings introduced a number of new issues, see bug 1572564 for the visual ones. Also, this panel now screams to be browseable via screen reader reading/browse mode commands. So, to address these:

  1. All of the panels should be role "document", not just some of them. There is enough information in here that is in textual form only, not part of a control, that it makes sense to give screen reader users the ability to browse through them with their familiar reading commands.
  2. Consequently, make some of the labels ARIA headings so they can be navigated to more easily.
  3. The new more-info button is missing an accessible (localizable) label.

Patches in the works.

Summary: More accessibility fix-ups for screen re → More accessibility fix-ups for screen readers

While working on this, found a bug in the core accessibility engine that label associations don't generate the accessible name, although the association itself does work, e. g. clicking the label sets focus to the input or textarea.

Depends on: 1572677
Priority: -- → P1
Whiteboard: [skyline] → [privacy-panel][skyline]
Pushed by mzehe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a90563d008f4
Part 1: Make all of the privacy panel sub panels browseable documents for screen readers, r=johannh
https://hg.mozilla.org/integration/autoland/rev/526168f22912
Part 2: Make some of the section headers into headings for screen readers, r=johannh
https://hg.mozilla.org/integration/autoland/rev/84c3e4feee7b
Part 3: Associate the labels to their text inputs in the Report Site Not Working panel, r=johannh
https://hg.mozilla.org/integration/autoland/rev/2aa913a0c7ee
Part 4: Give the More Info button a label for screen readers, r=johannh
No longer depends on: 1572677
Regressions: 1583826
You need to log in before you can comment on or make changes to this bug.