Closed
Bug 1335734
Opened 8 years ago
Closed 8 years ago
ARIA role "directory" needs to expose the READONLY state
Categories
(Core :: Disability Access APIs, defect)
Core
Disability Access APIs
Tracking
()
RESOLVED
FIXED
mozilla54
| Tracking | Status | |
|---|---|---|
| firefox54 | --- | fixed |
People
(Reporter: MarcoZ, Assigned: MarcoZ)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Right now, it doesn't, causing NVDA to skip its contents in the virtual buffer and treat it as a widget. This page https://taz.de/Politik/Deutschland/!p4616/ exposes that problem.
According to spec[1], role="directory" inherits from role="list" and thus should also expose the READONLY state like "list" does.
[1] https://www.w3.org/TR/wai-aria/roles#directory
|
||
Comment 1•8 years ago
|
||
Add states::READONLY into directory role mapping in ARIAMap.cpp, add a proper test into /states/test_aria.html
Keywords: good-first-bug
Whiteboard: [mentor=:surkov]
|
||
Comment 2•8 years ago
|
||
Note that the ARIA mapping guide says ROLE_SYSTEM_LIST without read-only:
https://www.w3.org/TR/wai-aria-implementation/#mapping_role_table
However, I'd argue this is a bug in the guide given that it's a subclass of list, not listbox.
|
Assignee | |
Comment 3•8 years ago
|
||
I am actually taking this one, since I already have a patch. I use that news site daily now, and the bug annoys the hell out of me. ;)
(In reply to James Teh [:Jamie] from comment #2)
> Note that the ARIA mapping guide says ROLE_SYSTEM_LIST without read-only:
> https://www.w3.org/TR/wai-aria-implementation/#mapping_role_table
> However, I'd argue this is a bug in the guide given that it's a subclass of
> list, not listbox.
I agree. Jamie, are you going to file a bug against the spec?
| Comment hidden (mozreview-request) |
|
|
Assignee | |
Comment 5•8 years ago
|
||
| treeherder | ||
|
|
||
Comment 6•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8832770 [details]
Bug 1335734 - Make sure ARIA role 'directory' exposes the READONLY state so it doesn't get treated as a widget,
https://reviewboard.mozilla.org/r/109012/#review110300
Attachment #8832770 -
Flags: review?(surkov.alexander) → review+
|
|
||
Comment 7•8 years ago
|
||
i open an issue at core-aaam https://github.com/w3c/aria/issues/520
Pushed by mzehe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/13a04dc6650c
Make sure ARIA role 'directory' exposes the READONLY state so it doesn't get treated as a widget, r=surkov
|
||
Comment 9•8 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in
before you can comment on or make changes to this bug.
Description
•