tbody in Accessibility Tree gets focused when opening the context menu
Categories
(DevTools :: Accessibility Tools, defect)
Tracking
(firefox-esr60 unaffected, firefox64 unaffected, firefox65 wontfix, firefox66 fixed)
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox64 | --- | unaffected |
firefox65 | --- | wontfix |
firefox66 | --- | fixed |
People
(Reporter: Jamie, Assigned: yzen)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
STR (with NVDA):
- Open any document; e.g. data:text/html,foo
- Open the page context menu and select Inspect Accessibility Properties.
- Focus the Accessibility Tree.
- Use the arrow keys to select a node.
- Right click on the node to open the context menu.
Expected: NVDA should say "menu".
Actual: NVDA says "text frame" then "menu". - Press escape to dismiss the menu.
Expected: NVDA should speak the item selected in step 4.
Actual: NVDA speaks "text frame" instead of the item.
I think the tbody gets DOM focus when you click in the tree view. It does have role="presentation", but we ignore that for nodes which can get DOM focus. Assuming I'm correct (I haven't looked at the code), there are two possible solutions:
- The tbody shouldn't be focusable. This has the added benefit of pruning the tbody from the a11y tree. While the ideal solution, I'm guessing it had to be made focusable for some implementation reason.
- Set aria-activedescendant on the tbody to also reference the active tree item. This way, when the tbody does get focus, a11y clients won't really notice it.
Comment 1•6 years ago
|
||
This used to work fine, so I'm marking this as a regression.
Assignee | ||
Comment 2•6 years ago
|
||
MozReview-Commit-ID: 2kzvA2Eb1eF
Assignee | ||
Updated•6 years ago
|
Comment 4•6 years ago
|
||
Backout for browser_inspector_extension_sidebar.js, dt failure, on a CLOSED TREE
Backout log: https://hg.mozilla.org/integration/autoland/rev/1c7ed5d9e1fe429f8a753b40156744b73ef12cda
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=222829598&repo=autoland&lineNumber=12535
Comment 6•6 years ago
|
||
Backed out changeset 50cc5c5bd5ef (Bug 1520107) for mochitest failures at test_accessible_row_context_menu.html
Backout: https://hg.mozilla.org/integration/autoland/rev/1aee4f577b05824f71ca36db2ab640e76d90cc60
Push that started the failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=pending%2Crunning%2Csuccess%2Ctestfailed%2Cbusted%2Cexception&revision=50cc5c5bd5efdb15adda6b1490ecf907f5fe100b&selectedJob=222885265
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=222885265&repo=autoland&lineNumber=2349
Comment 8•6 years ago
|
||
bugherder |
Assignee | ||
Updated•6 years ago
|
Comment 9•6 years ago
|
||
Too late for 65.
Description
•