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 accessibility performance #93

Closed
jendiamond opened this issue Jan 24, 2019 · 1 comment
Closed

Add accessibility performance #93

jendiamond opened this issue Jan 24, 2019 · 1 comment

Comments

@jendiamond
Copy link

jendiamond commented Jan 24, 2019

There are some accessibility issues with this gem. I am working on some fixes now but they aren't perfected yet. :)


The date range slider form accessibility issues on our site

The form tag is not labelled but the slider is well identified and useable. I added an aria-labelledby to the partial _range_limit_pane in our site but I think it needs to be a bit different.

Screen Shot 2019-01-24 at 11.03.09 AM.png

<div class="limit_content range_limit" aria-labelledby="range limit slider">

Possible fixes

@jrochkind
Copy link
Member

Main branch and forthcoming 9.0 release has significantly revamped UI.

It no longer features a slider at all, so this specific issue will no longer be relevant.

It has various other tweaks intended to improve accessibility. If any problems still remain, please open another issue!

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

No branches or pull requests

2 participants