1608883 - Support multiple aria-details ids

Status: RESOLVED FIXED

(Core :: Disability Access APIs, enhancement, P2)

Description

[Aaron Leventhal]

As part of work on ARIA Annotations, the ARIA spec is being updated to allow aria-details to support multiple ids. See https://github.com/w3c/aria/pull/1136
The plan has been approved by screen reader developers as well as the ARIA WG.
(If it's not in yet, it's because we are still working on the wording).

Comment 1

[Aaron Leventhal]

Note: ariaDetailsElement should be changed to ariaDetailsElements in AOM.
Filed https://github.com/WICG/aom/issues/157 for AOM issue.

Comment 2

[Marco Zehe (:MarcoZ)]

Attachment: Bug 1608883 - Add test for multiple aria-details relations, r=jamie

Comment 3

[Marco Zehe (:MarcoZ)]

Our code already supports multiple target IDs for aria-details and is, strictly speaking, not spec compliant. But since this is wanted anyway, I merely added a test to test for multiple targets.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/705b7ee46fa6
Add test for multiple aria-details relations, r=Jamie

Comment 5

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/705b7ee46fa6

Comment 6

[Thorin [:thorin]]

just noticed line 320 in test_general.html contains tabple caption rather than table... - it's not important, but the perfectionist in me is screaming :)

Comment 7

[Thorin [:thorin]]

^^ :facepalm: see how easy it is to make typos: i meant line 390

Comment 8

[Marco Zehe (:MarcoZ)]

Has a test in the test suite, doesn't need manual testing.

Comment 9

[Chris Mills [:cmills]]

Documentation complete; see https://github.com/mdn/sprints/issues/2939#issuecomment-607404699 for the full details.