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 role="button" to [data-slide="prev"],[data-slide="next"] #13559

Merged
merged 1 commit into from
May 9, 2014

Conversation

cvrebert
Copy link
Collaborator

@cvrebert cvrebert commented May 9, 2014

@bassettsj
Copy link
Contributor

@cvrebert Adding an aria-label="Next Item" or <span class="sr-only">Next Item</span> might be helpful here.

@cvrebert
Copy link
Collaborator Author

cvrebert commented May 9, 2014

Yeah, that's still on the to-do list.

@cvrebert cvrebert added this to the v3.2.0 milestone May 9, 2014
cvrebert added a commit that referenced this pull request May 9, 2014
add role="button" to [data-slide="prev"],[data-slide="next"] to improve accessibility
@cvrebert cvrebert merged commit 9013839 into master May 9, 2014
@cvrebert cvrebert deleted the carousel-role branch May 9, 2014 23:02
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