Closed Bug 1428433 Opened 4 years ago Closed 4 years ago

Reduce accessibility tree noise by hiding shared component container elements from accessibility.

Categories

(DevTools :: Accessibility Tools, enhancement)

57 Branch
enhancement
Not set
normal

Tracking

(firefox60 fixed)

RESOLVED FIXED
Firefox 60
Tracking Status
firefox60 --- fixed

People

(Reporter: yzen, Assigned: yzen)

References

Details

Attachments

(1 file)

There are a number of shared react components that currently expose container nodes as generic "section" accessibles in: Draggable.js, SplitBox.js, TreeHeader.js. We need to add role presentation wherever possible to flatten the accessibility tree a little bit.
Attached patch 1428433 patchSplinter Review
Attachment #8951256 - Flags: review?(odvarko)
Comment on attachment 8951256 [details] [diff] [review]
1428433 patch

Review of attachment 8951256 [details] [diff] [review]:
-----------------------------------------------------------------

Looks reasonable, thanks!
Honza
Attachment #8951256 - Flags: review?(odvarko) → review+
Component: Developer Tools: Shared Components → Developer Tools: Accessibility Tools
Pushed by yura.zenevich@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8d1e97e0569c
added role 'presentation' to shared component elements that do not have any semantics. r=Honza
https://hg.mozilla.org/mozilla-central/rev/8d1e97e0569c
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
Product: Firefox → DevTools
Assignee: nobody → yzenevich
You need to log in before you can comment on or make changes to this bug.