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

[accname] Explicitly state UAs must ignore “aria-label” for slots #2385

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sideshowbarker
Copy link
Contributor

This change updates the AriaLabel substep of the “Computation steps” section of the accname spec to explicitly state that the step must not be performed if the node being visited is a slot element.

See the discussion at w3c/accname#173. It seems a requirement to ignore the aria-label attribute in this case — or to ignore any ARIA attributes present on slot elements — may follow from normative requirements stated in some other spec.

Nevertheless, a specific requirement should be stated at point of use in the text for this step in accname spec itself.

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: https://wpt.fyi/results/accname/name/shadowdom/slot.html
  • Browser implementations (link to issue or commit): WebKit, Gecko, and Blink already implement this.
  • Does this need AT implementations? No
  • Related APG Issue/PR: N/A
  • MDN Issue/PR: N/A

This change updates the AriaLabel substep of the “Computation steps”
section of the “Accessible Name and Description Computation” spec to
explicitly state that the step must not be performed if the node being
visited is a slot element.
Copy link

netlify bot commented Nov 25, 2024

Deploy Preview for wai-aria ready!

Name Link
🔨 Latest commit aa1a22b
🔍 Latest deploy log https://app.netlify.com/sites/wai-aria/deploys/6744306015568d00086d6489
😎 Deploy Preview https://deploy-preview-2385--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 site configuration.

@jnurthen jnurthen requested a review from keithamus December 5, 2024 18:16
@jnurthen jnurthen added the Agenda label Dec 5, 2024
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.

2 participants