Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modals sollten barrierefrei umgesetzt sein #136

Open
4 tasks
fuenkchen opened this issue Jun 20, 2022 · 1 comment
Open
4 tasks

Modals sollten barrierefrei umgesetzt sein #136

fuenkchen opened this issue Jun 20, 2022 · 1 comment

Comments

@fuenkchen
Copy link
Contributor

fuenkchen commented Jun 20, 2022

  • Das Login-Modal ist nicht barrierefrei umgesetzt. Für Screenreader-User bleibt der Inhalt hinter dem Modal weiterhin erreichbar.
  • iOS: Beim Öffnen der Navigation liegt der Fokus weiterhin im Hintergrund, so dass die Elemente "Magazin", "Feed" etc. im Hintergrund mit VoiceOver angesteuert werden und nicht die Elemente in der Navigation.
  • iOS: Bei der der Lightbox/vergrösserten Ansicht von Bildern können die Elemente im Hintergrund weiterhin mit VoiceOver erreicht werden.
  • Das Dropdown-Menü bei Serien Die Serien-Navi hat keinen Status, so dass es für Screenreader-User nicht erkennbar ist, ob das Menü ein- oder ausgeklappt ist.

Details in den Issues Nr. 63 - 66 im Accessibility Review

@fuenkchen fuenkchen added this to the Accessibility Review milestone Jun 20, 2022
@tpreusse tpreusse changed the title Dialog (Modals) & Dropdown-Menüs sollten barrierefrei umgesetzt sein Modals sollten barrierefrei umgesetzt sein Jun 28, 2022
@trm217
Copy link
Contributor

trm217 commented Aug 5, 2022

Using focus-trap-react (react implementation of focus-trap) on the modal & lightbox might be a good way to solve the mentioned problems

@trm217 trm217 self-assigned this Aug 5, 2022
@trm217 trm217 linked a pull request Aug 5, 2022 that will close this issue
9 tasks
@trm217 trm217 removed their assignment Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants