Improve A11y support
cassiozen opened this issue · 2 comments
cassiozen commented
Hi, I want to help with a PR for a11y - here are the things I would like to tackle:
- Change
getHandleProps
to return by default:role
('slider'),aria-valuemin
,aria-valuemax
andaria-valuenow
. - Change
getHandleProps
to not settabindex
and not setoutline
to none. Ideally, update the examples to useButtons
for handles (since they already are interactive elements and have a tab stop set by default). - Add support for keyboard navigation (When a handle is focused, the user should be able to use left/right arrows to change value)
Do you think these changes make sense for the library?
tannerlinsley commented
Absolutely! Go for it :)