KEMBAR78
Fix incorrect role specified for html element - 571 by scottaohara · Pull Request #2504 · w3c/aria · GitHub
Skip to content

Conversation

scottaohara
Copy link
Member

@scottaohara scottaohara commented Apr 1, 2025

closes w3c/html-aam#571

changes the listed mapping for the html element to be the generic role, rather than the document role.

A note is added to clarify the document role of a web page is exposed by a parent document node created by the user agent.

From what i can tell, no implementation bugs need to be filed, as browsers have been doing the right thing - and this is a spec correction to match reality.


Preview | Diff

closes #571

changes the listed mapping for the html element to be the generic role, rather than the document role.

A note is added to clarify the document role of a web page is exposed by a parent `document node` created by the user agent.
@netlify
Copy link

netlify bot commented Apr 1, 2025

Deploy Preview for wai-aria ready!

Name Link
🔨 Latest commit af4cdc1
🔍 Latest deploy log https://app.netlify.com/projects/wai-aria/deploys/683eede49f072a0008314f31
😎 Deploy Preview https://deploy-preview-2504--wai-aria.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

scottaohara added a commit to w3c/html-aria that referenced this pull request Apr 1, 2025
per w3c/aria#2504 - the html element doesn't actually map to the `document` role - rather a parent `document node` to the `html` element is what exposes the `document` role of a web page.

This update fixes the spec to match the updated HTML aam mapping / matches reality.  

The `generic` role is being added as an "allowed" but not recommended role to the `html` element.  The `document` role is being kept as an allowed role for the `html` element.  The rational there being that this has been in the spec for a long time and even when specified on the html element, from what i can tell, there's no pro/con to its inclusion.
@keithamus keithamus self-requested a review April 3, 2025 17:19
@jnurthen jnurthen requested a review from cookiecrook April 3, 2025 17:21
Copy link
Contributor

@cookiecrook cookiecrook left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Blocking the other approved reviews until we discuss this side effect of the change.

@scottaohara scottaohara requested a review from cookiecrook April 4, 2025 14:29
@spectranaut spectranaut moved this from All PRS to Needs Consensus/Review in ARIA Normative PR Tracking Jun 13, 2025
@spectranaut
Copy link
Contributor

@cookiecrook it sounds like your issue was resolved, can you positively review this?

@pkra
Copy link
Member

pkra commented Jun 24, 2025

I've edited the top card to point to the html-aam issue.

@scottaohara scottaohara merged commit df5d60a into main Jun 27, 2025
10 checks passed
@github-project-automation github-project-automation bot moved this from Needs Review to Waiting For Implementation in ARIA Normative PR Tracking Jun 27, 2025
github-actions bot added a commit that referenced this pull request Jun 27, 2025
SHA: df5d60a
Reason: push, by scottaohara

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@scottaohara scottaohara deleted the htmlaam-571 branch June 27, 2025 11:22
scottaohara added a commit to w3c/html-aria that referenced this pull request Jul 23, 2025
Correction for HTML element role mapping

per w3c/aria#2504 - the html element doesn't actually map to the `document` role - rather a parent `document node` to the `html` element is what exposes the `document` role of a web page.

This update fixes the spec to match the updated HTML aam mapping / matches reality.  

The `generic` role is being added as an "allowed" but not recommended role to the `html` element.  The `document` role is being kept as an allowed role for the `html` element.  The rational there being that this has been in the spec for a long time and even when specified on the html element, from what i can tell, there's no pro/con to its inclusion.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

html-aam uses incorrect mapping for html element

6 participants