Skip to content

editorial: Fix the role attribute definition#2798

Open
jnurthen wants to merge 8 commits into
mainfrom
2145
Open

editorial: Fix the role attribute definition#2798
jnurthen wants to merge 8 commits into
mainfrom
2145

Conversation

@jnurthen
Copy link
Copy Markdown
Member

@jnurthen jnurthen commented May 27, 2026

🚀 Netlify Preview:
🔄 this PR updates the following sspecs:

Closes #2145,

export attr-role, MUST allow, glossary data-noexport, role-attribute term linking.
related work: whatwg/html#6890.

And a few other todo items (delete this section after performing them):

  • For every spec that this PR edits, please add the appropriate spec:<spec_name> label. If you don't have privileges to do this, editors will do it for you.
  • If the change is editorial, please add "Editorial:" at the start of your PR name, and delete the "Test, Documentation and Implementation tracking" section below.

Test, Documentation and Implementation tracking

Once this PR has been reviewed and has consensus from the working group, tests should be written and issues should be opened on browsers. Add N/A and check when not applicable.

  • "author MUST" tests: N/A
  • "user agent MUST" tests: N/A
  • Browser implementations (link to issue or commit):
    • WebKit: no change needed
    • Gecko: no change needed
    • Blink: no change needed
  • ACT review? N/A
  • Does this need AT implementations? N/A
  • Related APG Issue/PR: N/A
  • MDN Issue/PR: N/A

@netlify
Copy link
Copy Markdown

netlify Bot commented May 27, 2026

Deploy Preview for wai-aria ready!

Name Link
🔨 Latest commit f42af65
🔍 Latest deploy log https://app.netlify.com/projects/wai-aria/deploys/6a19e4987e1d1d00080b8c9d
😎 Deploy Preview https://deploy-preview-2798--wai-aria.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 27, 2026

🚀 Deployed on https://deploy-preview-2798--wai-aria.netlify.app

@github-actions github-actions Bot temporarily deployed to pull request May 27, 2026 18:28 Inactive
Comment thread index.html Outdated
Comment thread index.html Outdated
Comment thread index.html Outdated
Copy link
Copy Markdown
Contributor

@spectranaut spectranaut left a comment

Choose a reason for hiding this comment

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

This looks good to me, and I think it does help solve the reported issue. I agree with all of @adampage's requests for changes/clarity, if they are possible :)

Comment thread index.html Outdated
…-link Role Attribute spec

- Restore <code>role</code> inside all <a>role attribute</a> link references
  throughout the document, per adampage's review suggestions
- Deep-link the Role Attribute 1.0 spec reference to the specific
  #s_role_module_attributes section

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jnurthen jnurthen changed the title Fix the role attribute definition editorial: Fix the role attribute definition May 29, 2026
@github-actions github-actions Bot temporarily deployed to pull request May 29, 2026 12:39 Inactive
@github-actions github-actions Bot temporarily deployed to pull request May 29, 2026 12:56 Inactive
@github-actions github-actions Bot temporarily deployed to pull request May 29, 2026 13:23 Inactive
@github-actions github-actions Bot temporarily deployed to pull request May 29, 2026 19:00 Inactive
@github-actions github-actions Bot temporarily deployed to pull request May 29, 2026 19:11 Inactive
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.

Fix the role attribute definition.

5 participants