-
-
Notifications
You must be signed in to change notification settings - Fork 16
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
Is there a way to control the slider position using the keyboard? #31
Comments
Hi @lucianourgal, thanks for opening this issue! The library doesn't currently provide keyboard bindings but I'm definitely onboard with implementing some functionality or accepting a PR. I looked into it briefly before but was unsure on how to make the component properly accessible (with the correct aria tags etc.). With the current version it's possible to implement your own keyboard functionality by adding a Here's a super basic demo listening to the |
Thank you @nerdyman, it worked! |
Awesome! Thanks for opening the issue, I'll see if there's a clean way to add this to library in future. |
Apologies for commenting on a closed issue, but just wanted to suggest that alternative way to do this would be to include an HTML 'range' I'm going to be using this component on a project I'm working on which will need to have accessibility support - so I plan to try something along these lines. Really nice component btw, thanks for making this available. |
That's a good shout @moloko! Let me know how it goes, I'm totes up for improving accessibility. |
This has been released in v3! npm install react-compare-slider@latest
yarn add react-compare-slider@latest
pnpm install react-compare-slider@latest |
This would be interesting because of accessibility issues.
The text was updated successfully, but these errors were encountered: