Accessibility requirements

banthagroup / fslightbox

An easy to use vanilla JavaScript plug-in without production dependencies for displaying images, videos, or, through custom sources, anything you want in a clean overlying box.

MIT License

351 stars 30 forks source link

Accessibility requirements #147

Open harrync opened 3 years ago

harrync commented 3 years ago

Hiya,

Using the JS Pro version of the library, am noting some accessibility issues and wondering if there is anything on your roadmap to address these?

Some things I've noticed -

Lightbox buttons are div elements rather than buttons.
No tab navigation on lightbox when opened, tabbing continues in the rest of the page
Caption/description isn't picked up by audio description software

Cheers Harry

MathieuAlphamosa commented 3 years ago

I concur, this is the only issue stopping me from using fslightbox.

salientknight commented 3 years ago

I'm in the same boat. The controls for the lightbox need a tab index. Keyboard controls work for going through slides but not for tabbing through controls.

I added tabindex and roll:button — and tabbing still does not work. The tabbing actually tabs through the browser tabs, which it should not be accessing without a ctrl-tab being used. Even with the tabindexes tabbing never reaches the menu. Left and Right arrows work for moving through the images and I was able to get Enter to work, but I think that's a carryover from the page below as I can see it tabbing through my images grid even though it's covered with the lightbox.

mandrasch commented 10 months ago

hi, any updates on this? 🙂 (especially important for EU websites to prepare for 2025 https://kinsta.com/blog/european-accessibility-act/)

lubo-makky commented 3 weeks ago

Hi, we are using vanilla JS PRO version and would also very much appreciate if accessibility could be addressed.