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

Add aria-controls attribute to input element #17257

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

weeblr
Copy link
Contributor

@weeblr weeblr commented Jul 24, 2024

Currently using aria-owns attribute to relate input and listbox, per ARIA 1.0. Added use of aria-controls, per ARIA 1.1.
Left aria-owns in place for B/C and also makes sense as list is a sibling of input anyway.

Currently using aria-owns attribute to relate input and listbox, per ARIA 1.0. Added use of aria-controls, per ARIA 1.1. Left aria-owns in place for B/C and also makes sense as list is a sibling of input anyway.
Copy link
Owner

@LeaVerou LeaVerou left a comment

Choose a reason for hiding this comment

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

Thank you!

@LeaVerou LeaVerou merged commit 04f0f6e into LeaVerou:gh-pages Jul 24, 2024
@weeblr
Copy link
Contributor Author

weeblr commented Jul 24, 2024

@LeaVerou Thanks for merging!

Will this be in an update to the npm package or is it only distributed through https://projects.verou.me/awesomplete/?

@LeaVerou
Copy link
Owner

I guess I should publish it, it's been a long time!

@weeblr
Copy link
Contributor Author

weeblr commented Jul 25, 2024

That'd be great, thank you!

@LeaVerou
Copy link
Owner

Done!

Wow this package has some old stuff. bower.json lol, blast from the past 🤣

@weeblr
Copy link
Contributor Author

weeblr commented Jul 25, 2024

Thanks for taking the time to update the package, it'll be useful!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants