Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorporate host-language unique computedrole string for reserved native elements with no direct ARIA equivalent #511

Closed
Tracked by #70 ...
cookiecrook opened this issue Oct 17, 2023 · 5 comments · Fixed by #519
Assignees

Comments

@cookiecrook
Copy link
Collaborator

Incorporate Example 3 in https://w3c.github.io/core-aam/#roleMappingComputedRole

When host language elements do not have an exact or equivalent mapping to a valid, non-abstract role, the related Accessibilty API Mapping extension specification MAY specify a unique computedrole string as the return value for interoperability testing purposes, such as <video> -> "html-video" in [HTML-AAM]. However, authors MUST NOT use any host-language-prefixed computedrole string in the role attribute (such as html-video), unless the token also matches valid, defined role (such as dpub-chapter). User Agents MUST ignore any abstract or invalid role token.

EXAMPLE 3

<video> <!-- computedrole returns "html-video" -->
<main role="html-video"> <!-- Author error. computedrole returns "main" -->
@cookiecrook
Copy link
Collaborator Author

I thought this one existed already, but I could not find it in HTML-AAM. I was probably thinking of a related tracker in the main ARIA spec repo.

scottaohara added a commit that referenced this issue Nov 22, 2023
closes #511

Incorporates a version of the `computedrole` content from core aam, and re-adds all the instances of computed roles which were accidentally dropped when changing the spec from one giant table with details/summary plugin to individual tables.
@scottaohara
Copy link
Member

Looks like the computed role content got deleted when HTML AAM was converted form the details/summary/table layout to just tables.

I'll get these all back into the spec with the PR to address this.

@spectranaut
Copy link
Contributor

Ugh, sorry @scottaohara, I guess that was my fault :(

@scottaohara
Copy link
Member

No more your fault than mine, since I didn’t catch it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants