Skip to content

Update accessibility guidelines for tab and tabpanel#2690

Merged
pkra merged 5 commits intomainfrom
follow-on-to-2500
Feb 11, 2026
Merged

Update accessibility guidelines for tab and tabpanel#2690
pkra merged 5 commits intomainfrom
follow-on-to-2500

Conversation

@scottaohara
Copy link
Copy Markdown
Member

@scottaohara scottaohara commented Dec 2, 2025

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

PR to add duplication of requirements between tab and tabpanel - see #2500 (comment)

note i also duplicated the author should requirement for programmatically associating the tab with the tabpanel, since that seems important to call out these options re: validator/validator#1817


Preview | Diff

PR to add duplication of requirements between tab and tabpanel - see #2500 (comment)

note i also duplicated the author should requirement for programmatically associating the tab with the tabpanel, since that seems important to call out these options re: validator/validator#1817
@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 2, 2025

Deploy Preview for wai-aria ready!

Name Link
🔨 Latest commit 0b49dbe
🔍 Latest deploy log https://app.netlify.com/projects/wai-aria/deploys/69725cd9d131ef0008391447
😎 Deploy Preview https://deploy-preview-2690--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.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR duplicates accessibility requirements between the tab and tabpanel role definitions to ensure consistency in documentation. The changes align with W3C ARIA issue #2500 and validator issue #1817, ensuring that both roles document the MUST requirement for rendering active tabs/panels and the SHOULD requirement for programmatic association.

  • Adds the active tab/tabpanel rendering requirement to both role definitions
  • Documents association methods using aria-controls and aria-labelledby in both sections
  • Adds HTML comments to mark synchronized content blocks

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread index.html Outdated
Comment thread index.html Outdated
Comment thread index.html Outdated
Comment thread index.html
Comment thread index.html
Comment thread index.html Outdated
Comment thread index.html Outdated
Comment thread index.html
Copy link
Copy Markdown
Contributor

@accdc accdc left a comment

Choose a reason for hiding this comment

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

Sounds good to me. :)

scottaohara and others added 3 commits December 8, 2025 12:16
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

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

@github-actions github-actions Bot temporarily deployed to pull request January 22, 2026 17:23 Inactive
@pkra pkra moved this to Merge? in ARIA Normative PR Tracking Jan 22, 2026
@pkra pkra added the editorial a change to an example, note, spelling, grammar, or is related to publishing or the repo label Feb 11, 2026
@pkra pkra merged commit 16a714f into main Feb 11, 2026
9 checks passed
@github-project-automation github-project-automation Bot moved this from Merge? to Waiting For Implementation in ARIA Normative PR Tracking Feb 11, 2026
@spectranaut spectranaut moved this from Waiting For Implementation to Remaining implementations (post merge) in ARIA Normative PR Tracking Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial a change to an example, note, spelling, grammar, or is related to publishing or the repo spec:aria

Projects

Status: No status
Status: Remaining implementations (post merge)

Development

Successfully merging this pull request may close these issues.

7 participants