1796178 - Request to revise mapping for <s> element

Status: RESOLVED FIXED

(Core :: Disability Access APIs, enhancement)

Description

[scottaohara]

Per HTML AAM PR 442 - https://github.com/w3c/html-aam/pull/442

It was decided by the ARIA working group that it made sense to revise the mapping of HTML's <s> element to be exposed as an implicit role=deletion.

It would be expected that the following three markup examples all be exposed with this role:

<del>foo</del>
<s>foo</s>
<span role=deletion>foo</span>

HTML AAM defers to Core AAM for how the deletion role is to be mapped - https://w3c.github.io/core-aam/#role-map-deletion

Comment 1

[James Teh [:Jamie]]

Implementation guidance:
This should be done similar to https://hg.mozilla.org/mozilla-central/rev/b6c8ed7e23e3. However, you don't need to add any new roles here because CONTENT_INSERTION/DELETION already exist. You just need to add the markup mapping and the test.

Comment 2

[Anna Yeddi [:ayeddi]]

Attachment: Bug 1796178 - Revise mapping for <s> element to role=deletion. r=Jamie

Updated HTMLMarkupMap to expose <s> element with an implicit roles::CONTENT_DELETION, updated accessible/tests/mochitest/elm/test_HTMLSpec.html Mochitest and expected results for testing/web-platform/tests/html-aam/roles.html wpt test

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1d562ee8729b
Revise mapping for <s> element to role=deletion. r=Jamie

Comment 4

[Norisz Fay [:noriszfay]]

Backed out for causing mochitest failures on test_general.html

Backout link

Push with failures

Failure log

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/38223a5468bb
Revise mapping for <s> element to role=deletion. r=Jamie

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/38223a5468bb