-
Notifications
You must be signed in to change notification settings - Fork 125
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 PR 183] Step 2C includes controls with value that are directly referenced by labelledby or describedby #2210
base: main
Are you sure you want to change the base?
Conversation
…ect current browser behavior
This move was a bit messier since the PR is a year old and had merge conflicts. I fixed two details which I hope are editorial:
@smhigley could you please double check when you find the time? |
dc15b14
to
405fb4f
Compare
Ugh - I had forgotten to stage the fixed merge conflicts. I've force pushed the correct merge now. |
Now pointing to main. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Thanks for porting this over, @pkra! The ol > ul change actually wasn't part of the original committed change, so I think I'll remove that part and just leave the change in wording. I think it got introduced in resolving merge conflicts somewhere 😅 |
✅ Deploy Preview for wai-aria ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Moved from w3c/accname#183