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.

Sign up for GitHub

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

Jump to bottom

Carousel component could be made more accessible. #1484

Closed
nmggithub opened this issue Mar 7, 2024 · 2 comments · Fixed by #1516
Closed

Carousel component could be made more accessible. #1484

nmggithub opened this issue Mar 7, 2024 · 2 comments · Fixed by #1516
Labels
enhancement New feature or request

Comments

@nmggithub
Copy link
Contributor

Description

According to the W3C ARIA Authoring Practices Guide, specifically the page on the Carousel pattern using tab-based ARIA roles is one of the suggested uses of ARIA. The Carousel component in Nuxt UI does not use this pattern. While the APG is not normative, I do think this suggestion should be considered.

Additional context

No response
@nmggithub nmggithub added the enhancement New feature or request label Mar 7, 2024
@benjamincanac Volta.net
Copy link
Member

Would you mind opening a PR for this?

@nmggithub
Copy link
Contributor Author

nmggithub commented Mar 11, 2024 •
edited
Loading

I could definitely do so! I would have to do some thinking regarding some of the patterns that Carousel uses (particularly when there are multiple items displayed).

@nmggithub nmggithub mentioned this issue Mar 14, 2024
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(Carousel): add tab-based ARIA roles nmggithub/ui
2 participants
@benjamincanac @nmggithub