Closed Bug 1569868 Opened 1 year ago Closed 8 months ago

Inspect Accessibility Properties doesn't focus accessibility tree

Categories

(DevTools :: Accessibility Tools, defect)

defect
Not set
normal

Tracking

(firefox72 fixed, firefox73 fixed)

RESOLVED FIXED
Firefox 73
Tracking Status
firefox72 --- fixed
firefox73 --- fixed

People

(Reporter: Jamie, Assigned: yzen)

Details

Attachments

(1 file)

STR:

  1. Open https://www.mozilla.org/
  2. Right click the Mozilla link (or tab to it and press shift+f10) and select Inspect Accessibility Properties.
    • Expected: The Accessibility Tree widget should get focus.
    • Actual: The Accessibility Panel gets focus, but not the tree.
  3. Without closing the panel, right click on the Get a Firefox Account link (or tab to it and press shift+f10) and select Inspect Accessibility Properties.
    • Expected: The Accessibility Tree widget should get focus.
    • Actual: The Accessibility Panel gets focus, but not the tree.

Impact: This means the user has to tab a few times to get to the tree, which is rather tedious and inefficient if you use this a lot.
Note that the main inspector focuses the markup tree as expected.

Assignee: nobody → yzenevich
Status: NEW → ASSIGNED
Pushed by yura.zenevich@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/5fd7ad835de6
ensure that accessibility properties list is focused when using browser or inspector context menus to inspect accessibility properties. r=mtigley
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 73

Comment on attachment 9114920 [details]
Bug 1569868 - ensure that accessibility properties list is focused when using browser or inspector context menus to inspect accessibility properties. r=mtigley

Beta/Release Uplift Approval Request

  • User impact if declined: Without this patch, screen reader and keyboard users will have keyboard focus shifted into a position that is confusing. Users expect to be focused on the list of accessibility properties and not the panel document.
  • Is this code covered by automated tests?: Yes
  • 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): Simply ensuring that focus shifts onto the list of accessibility properties when using context menu to inspect accessibility.
  • String changes made/needed: None
Attachment #9114920 - Flags: approval-mozilla-beta?

This bug was untouched for 5 months, so presumably this isn't a new issue; does something make it more urgent now?

Flags: needinfo?(yzenevich)

(In reply to Julien Cristau [:jcristau] from comment #5)

This bug was untouched for 5 months, so presumably this isn't a new issue; does something make it more urgent now?

This is serious issue for screen reader / keyboard users that just hasn't been addressed in timely manner.

Flags: needinfo?(yzenevich)

Comment on attachment 9114920 [details]
Bug 1569868 - ensure that accessibility properties list is focused when using browser or inspector context menus to inspect accessibility properties. r=mtigley

small fix for devtools a11y inspector, approved for 72.0b7

Attachment #9114920 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.