haltersweb / Accessibility

Accessible solutions to web widgets and applications
http://haltersweb.github.io/Accessibility/
MIT License
125 stars 28 forks source link

Carousel lentils should communicate if they're selected #21

Closed tryenc closed 7 years ago

tryenc commented 7 years ago

Carousel lentils currently receive visual treatment when they're selected (they turn red), however, a screen reader user has no indication that said lentil is selected. I spoke to David about this issue.

haltersweb commented 7 years ago

Hmmm... They were supposed to announce something (like panel 1 of 3). Let me see if there is code I never checked in.

haltersweb commented 7 years ago

I checked and the lentils do have screen reader text in a span within the lentil buttons:

image

But there certainly is a better way to do this using aria-label. I will change it to that instead of a span. Cleaner.

I will let you know when it's up.

haltersweb commented 7 years ago

Updated code using aria-label instead of span.screen-reader-text