rangeslider.js icon indicating copy to clipboard operation
rangeslider.js copied to clipboard

aria-* attributes for accessibility

Open sjelfull opened this issue 8 years ago • 4 comments

Could you look into adding better accessibility in the form of aria-labels?

sjelfull avatar Mar 01 '16 14:03 sjelfull

@sjelfull good point 😊 Do u have some specific suggestions?

My first starting point would be this.

andreruffert avatar Mar 01 '16 15:03 andreruffert

I was doing some research on sliders just now, and it looks Airbnb is doing some nice stuff. I also bet they have have put more efforts into researching the actual accessibility beyond just reasing about aria-labels.

You can look more closely at their markup in their search (the price from-to range slider), but here is an excerpt:

<div aria-valuemax="58117"
     aria-valuemin="1300"
     aria-valuenow="1300" tabindex="1"></div>

<div aria-valuemax="150000"
     aria-valuemin="1300"
     aria-valuenow="58117" tabindex="2"></div>

sjelfull avatar Mar 01 '16 15:03 sjelfull

I just pull requested something very similar… Note that tabindex values in a11y recommendations can only be 0 or -1

@andreruffert I missed your aria-orientation work with my PR 😓

iManu avatar Nov 15 '16 14:11 iManu

Hi Team,

Range slider gives accessibility error on pa11y version 4.13.0. Kindly help me resolve the error as there are more components on my page using the slider hence more error.

pa11y-error

Bindhyachal avatar Feb 09 '18 07:02 Bindhyachal