search

akiran / react-slick

React carousel component
http://react-slick.neostack.com/
MIT License
11.72k stars 2.1k forks source link

sets always slide tabindex to -1 #1462

Open akashmdkml opened 5 years ago

akashmdkml commented 5 years ago

Is there any way to overwrite tabIndex to make slides accessible. please refer my use case https://codesandbox.io/s/myqq0yz54y voiceover should read left-arrow --> current slide(active slide) --> next-arrow to achieve this we should set current slide tabIndex to 0 and rest all slides tabIndex to -1

LuigiClaudio commented 5 years ago

There's a problem here https://github.com/akiran/react-slick/blob/master/src/slider.js#L183 where the slide content gets a tabindex value of '-1' - Also I could see on track.js some hard-coded values for the tabindex set once again to '-1': https://github.com/akiran/react-slick/blob/master/src/track.js although I think those work as intended.

jons-disco commented 5 months ago

Why is it set to -1 anyways. This breaks accessibility and is a sticking point for us to not use this library.