Giters

Accessible360 / accessible-slick

the last (accessible) carousel you'll ever need.

Home Page:https://accessible360.github.io/accessible-slick

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Improve the individual slide container markup

jasonwebb opened this issue · comments

commented
  • Eliminate role="tabpanel" since these don't really look like tabs, especially when multiple slides are visible at a time.
  • Remove aria-hidden="false" from all visible slides, since false is the default value and all this really does is make AT work harder for no reason.
  • Remove tabindex="-1", since slides should never receive focus anyway.
  • Add role="group" to each slide
  • Add aria-label="slide".
  • Add useGroupRole setting that can be used to turn off / on the role="group" and aria-label.
  • Add documentation about this feature change in the main README
commented

Hi Jason, I am learning more about more accessible coding and have a question regarding aria-label="slide". Would it be better to have the ability to give each slide a name so it was more clear what the slide was depicting? When testing with my screen reader, although I gave a custom region label, there is no information when viewing this group and I need to dig deeper to learn about that slide.