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

fix: carousel should use ul/li for a11y #956

Merged
merged 1 commit into from Feb 10, 2019

Conversation

Projects
None yet
1 participant
@nolanlawson
Copy link
Owner

nolanlawson commented Feb 10, 2019

Feel kind of dumb for forgetting this, but a carousel is a list and so it should really use ul/li elements for better a11y.

@nolanlawson nolanlawson merged commit 2884955 into master Feb 10, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment