1569868 - Inspect Accessibility Properties doesn't focus accessibility tree

Status: RESOLVED FIXED

(DevTools :: Accessibility Tools, defect)

Description

[James Teh [:Jamie] (away returning 28 Jan)]

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.

Comment 1

[Yura Zenevich [:yzen] (please use "needinfo")]

Attachment: Bug 1569868 - ensure that accessibility properties list is focused when using browser or inspector context menus to inspect accessibility properties. r=mtigley

Comment 2

[Pulsebot]

Pushed by [e-mail]:
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

Comment 3

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/5fd7ad835de6

Comment 4

[Yura Zenevich [:yzen] (please use "needinfo")]

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

Comment 5

[Julien Cristau [:jcristau]]

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

Comment 6

[Yura Zenevich [:yzen] (please use "needinfo")]

(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.

Comment 7

[Julien Cristau [:jcristau]]

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

Comment 8

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/releases/mozilla-beta/rev/6873e096573c