metafizzy / flickity

:leaves: Touch, responsive, flickable carousels
https://flickity.metafizzy.co
7.53k stars 602 forks source link

Prev/Next arrows are not accessible. #1252

Open cpickhardt15 opened 2 years ago

cpickhardt15 commented 2 years ago

Hey there! I'm just writing to ask about accessibility. Folks who use keyboard navigation do not have an easy way to navigate between the arrows and the slides in a way that makes sense. The arrows come in the DOM after the slide content, when really it should read "Prev arrow, slide content, next arrow" for every instance. Is there a way that we could at least move the previous arrow to be in front of the slide content? Even just that small change would make more sense.

This generic test case showcases the issue: https://codepen.io/desandro/pen/azqbop

desandro commented 1 year ago

I see the W3G carousel example puts their controls before the content. I'll make the change