1778795 - Conditional role mapping of role=form element based on accessible name (WPT wai-aria/role/form-roles.html form without label)

Status: RESOLVED FIXED

(Core :: Disability Access APIs, enhancement)

Description

[spectranaut]

Steps to reproduce:

In OSX, open an html file that includes an unnamed form element in Firefox, such as:

<div role="form">Form contents.</div>

Use the accessibility inspect tools to see the AXSubrole of the role=form element.

Actual results:

The element has a landmark AXSubrole

Expected results:

The element should have no AXSubrole

See the relevant changes in CORE-AAM here: https://github.com/w3c/core-aam/pull/97

Comment 1

[spectranaut]

I only describe the bug for the OSX CORE-AAM mappings, but the others will have to be updated as well!

Comment 2

[James Teh [:Jamie]]

As a side note, Firefox already behaves like this for <form> without a name precisely to avoid the landmark cluttering described in https://github.com/w3c/core-aam/issues/11#issue-325089950, but we currently treat role="form" differently. I'm curious as to why this change is necessary for role="form", given that the problematic examples all covered issues with <form>, not role="form". Is it just consistency?

Comment 3

[James Teh [:Jamie]]

See also the discussion in https://github.com/w3c/core-aam/issues/100

Comment 4

[Nathan LaPré]

Attachment: Bug 1778795: role="form" with empty accessible name returns native role, r?Jamie

The ARIA spec requires that, for certain landmark roles with no author-specified
names, user agents must treat such elements as if no role had been provided.
This revision accomplishes that task by carving out an exception in
ARIATransformRole and ComputeARIARole for roles::FORM, similar to the existing
carveout for roles::REGION. This revision also implements a NameIsEmpty function
which is helpful for checking name emptiness (since we do this in a few places)
without leaving a "name" variable on the stack. Finally, this revision updates a
expected WPT failure (by removing it).

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b592c7990879
role="form" with empty accessible name returns native role, r=Jamie

Comment 6

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/b592c7990879