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

Allow aria-valuetext for use with combobox role #2417

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

scottaohara
Copy link
Member

@scottaohara scottaohara commented Jan 29, 2025

closes #2382
and is related to #2344 / #2369

This PR adds aria-valuetext as a supported property for the combobox role.

the aria-valuetext definition is slightly updated to indicate that the attribute can be used on other supported roles. I'm not sure if we want to do any further updates to that section in this PR - or if that really should be handled in a larger PR to resolve #711

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: we can't currently write tests
  • Browser implementations (link to issue or commit):
  • Does this need AT implementations?
  • Related APG Issue/PR:
  • MDN Issue/PR:

Preview | Diff

closes #2382
and is related to #2344 / #2369 

This PR adds aria-valuetext as a supported property for the combobox role.

the `aria-valuetext` definition is slightly updated to indicate that the attribute can be used on other supported roles.  I'm not sure if we want to do any further updates to that section in this PR - or if that really should be handled in a larger PR to resolve #711
Copy link

netlify bot commented Jan 29, 2025

Deploy Preview for wai-aria ready!

Name Link
🔨 Latest commit d8784b8
🔍 Latest deploy log https://app.netlify.com/sites/wai-aria/deploys/67a5159f2313310008029a15
😎 Deploy Preview https://deploy-preview-2417--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.

Copy link
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.

Looks good to me, one suggestion to clarify a SHOULD in the aria-valuetext description.

Co-authored-by: Valerie Young <[email protected]>
@scottaohara
Copy link
Member Author

Looks good to me, one suggestion to clarify a SHOULD in the aria-valuetext description.

thank you!

Copy link
Member

@adampage adampage left a comment

Choose a reason for hiding this comment

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

Looks great. 🚀

@pkra
Copy link
Member

pkra commented Feb 9, 2025

@scottaohara could you update the top card on what remains to do before merging this?

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.

Consider allowing aria-valuetext for combobox role aria-valuetext would be useful on many more roles
5 participants