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

lux-stepper-large: die weiter/zurück-Buttons des Steppers sollen auf "aria-disabled" umgestellt werden. #521

Open
patrowe opened this issue Aug 21, 2024 · 5 comments · May be fixed by #563
Assignees
Labels
a11y Enhancements or bugs regarding accessibility. enhancement New feature or request
Milestone

Comments

@patrowe
Copy link
Contributor

patrowe commented Aug 21, 2024

Aktuell sind die Navigations-Buttons disabled, bis ein Step korrekt abgeschlossen ist. Das soll geändert.
Der "Weiter"-Button soll auf Aria-Disabled umgestellt werden. Damit ist der Button immer klickbar und für Assistenztechnologien erreichbar. Ist der Step noch nicht abgeschlossen, wird ein "Fehlermeldung" geworfen, der von der Anwendung verwarbeitet werden muss und der nächste Step bleibt weiter blockiert.

@patrowe patrowe added enhancement New feature or request a11y Enhancements or bugs regarding accessibility. labels Aug 21, 2024
@patrowe patrowe added this to the 18.2.0 milestone Oct 30, 2024
@heli-Berry heli-Berry self-assigned this Nov 12, 2024
@heli-Berry
Copy link
Contributor

heli-Berry commented Nov 20, 2024

Es gibt für den Stepper eine neue Property: luxA11YMode. Wenn diese auf true gesetzt wird, hat luxDisabled im Stepper auf den Weiter und Finish Button keine Auswirkungen mehr. Der Button bekommt das Attribut aria-disabled und ist weiterhin "aktiv" und für Assistenztechnologien erreichbar. Wenn bei einem Fehler in dem Formular versucht wird weiter zu navigieren, wird ein Output Event geworfen, welches vom Programm entsprechend verarbeitet werden muss.

Testbeschreibung:

  • Wenn die Property luxA11YMode aktiviert wurde (Validierung muss aktiviert sein), erscheint bei einem nicht abgeschlossenen Step eine Textbox mit einem Error
  • Wenn der Step abgeschlossen wurde kann ohne Probleme weiter navigiert werden

@patrowe: Bitte testen

heli-Berry added a commit that referenced this issue Nov 20, 2024
… sollen auf "aria-disabled" umgestellt werden.
@heli-Berry heli-Berry assigned DSeifer and unassigned heli-Berry Nov 27, 2024
@heli-Berry heli-Berry assigned heli-Berry and unassigned DSeifer Nov 27, 2024
heli-Berry added a commit that referenced this issue Dec 10, 2024
heli-Berry added a commit that referenced this issue Dec 10, 2024
… sollen auf "aria-disabled" umgestellt werden.
heli-Berry added a commit that referenced this issue Dec 10, 2024
… sollen auf "aria-disabled" umgestellt werden.
@patrowe
Copy link
Contributor Author

patrowe commented Dec 12, 2024

Getestet unter MacOS Sequoia 15.1 mit den Browsern Safari, Chrome, Firefox in den jeweils aktuellsten Versionen
Testergebnis: OK

Getestet unter Windows 10 mit den Browsern Chrome, Firefox und Edge in den jeweils aktuellsten Versionen
Testergebnis: OK

@patrowe patrowe closed this as completed Dec 12, 2024
@patrowe patrowe reopened this Dec 12, 2024
@patrowe
Copy link
Contributor Author

patrowe commented Dec 12, 2024

Das Verhalten in der Anwendung entspricht den Anforderungen.
Das Aria-Label muss im Fehlerstatus des Steps auf "true" stehen und im Completed-Status des Steps auf "false".
Aktuelles Verhaltenbei luxA11yMode=true: aria-disabled=false
bei luxA11yMode=false: aria-disabled=true.

Das ist ein Fehler und muss korrigiert werden.

heli-Berry added a commit that referenced this issue Dec 12, 2024
… sollen auf "aria-disabled" umgestellt werden.
@heli-Berry
Copy link
Contributor

heli-Berry commented Dec 12, 2024

Das Fehler wurde korrigiert (es fehlte ein "!"). Jetzt wird wieder korrekt aria-disabled auf = true gesetzt wenn luxA11yMode = true ist. Außerdem wird das Aria-Label immer gesetzt. Wenn der Button enabled ist, ist das Label false, wenn der Button disabled ist, ist das Label true. Damit soll vermieden werden, dass wenn der luxA11yMode nicht benutzt wird, der Button von Screen Readern trotzdem als aktiv erkannt wird.

@patrowe: Bitte erneut testen

heli-Berry added a commit that referenced this issue Dec 12, 2024
… sollen auf "aria-disabled" umgestellt werden.
heli-Berry added a commit that referenced this issue Dec 12, 2024
… sollen auf "aria-disabled" umgestellt werden.
@patrowe
Copy link
Contributor Author

patrowe commented Dec 12, 2024

Der erneute Test hat ergeben, dass das Aria-Label jetzt korrekt gesetzt wird.

@patrowe patrowe closed this as completed Dec 12, 2024
@heli-Berry heli-Berry reopened this Dec 18, 2024
heli-Berry added a commit that referenced this issue Jan 2, 2025
… sollen auf "aria-disabled" umgestellt werden.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Enhancements or bugs regarding accessibility. enhancement New feature or request
Projects
None yet
3 participants