1335734 - ARIA role "directory" needs to expose the READONLY state

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Marco Zehe (:MarcoZ)]

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

[alexander :surkov (:asurkov)]

Add states::READONLY into directory role mapping in ARIAMap.cpp, add a proper test into /states/test_aria.html

Comment 2

[James Teh [:Jamie]]

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.

Comment 3

[Marco Zehe (:MarcoZ)]

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 4

[Marco Zehe (:MarcoZ)]

Attachment: Bug 1335734 - Make sure ARIA role 'directory' exposes the READONLY state so it doesn't get treated as a widget,

Review commit: https://reviewboard.mozilla.org/r/109012/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/109012/

Comment 5

[Marco Zehe (:MarcoZ)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=6fd424de305e0a92f63638665ae48e22e6b3e724

Comment 6

[alexander :surkov (:asurkov)]

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

Comment 7

[alexander :surkov (:asurkov)]

i open an issue at core-aaam https://github.com/w3c/aria/issues/520

Comment 8

[Pulsebot]

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

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/13a04dc6650c