Can't set tabIndex on handle

[gvincentlh]

We need to be able to disable tabbing into the rc-slider-handle by setting tabIndex="-1". Unfortunately it looks like tabIndex on the handle is currently hard-coded to "0", with no ability to override this value.

slider/src/Handle.jsx

Line 28 in 76d4cf3

tabIndex="0"

Could you please add the ability to override tabIndex on the handle?

[paranoidjk]

It's for keyboard accessibility, could you explain why you need to disable it ? #282

[gvincentlh]

@paranoidjk does the why really matter? We'd just like to have control over the tabIndex value on the handle. We want to create an explicit tab order on multiple page elements. Thanks.

[paranoidjk]

Then we maybe could add a api tabIndex: boolean | number (default to be 1, false to disable)

Would you like PR?

[gvincentlh]

@paranoidjk sounds good, yes will submit a PR tomorrow.

[gvincentlh]

@paranoidjk see the MDN docs on tabIndex here https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/tabindex

We should probably default it to "0" as it is now. Wouldn't want to break the existing behavior. But we should allow it to be overridden via props. I'll expose a tabIndex prop on both the Handle and Slider components. Slider would simply pass it through to Handle, so that we won't have to use a custom handle just to set tabIndex.