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.

Sign up for GitHub

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

Jump to bottom

9.4.1.2 - select: Name, Rolle, Wert verfügbar #7226

Closed
Roman-Use opened this issue Dec 19, 2024 · 3 comments · Fixed by #7454 or #7453
Closed

9.4.1.2 - select: Name, Rolle, Wert verfügbar #7226

Roman-Use opened this issue Dec 19, 2024 · 3 comments · Fixed by #7454 or #7453
Assignees
@anicyne
Labels
v2 v3 Here are issues that need to be resolved for version 3.

Comments

@Roman-Use
Copy link

Link to the code that reproduces this issue:
Kolibri Version 2.2.1

Which browser or operating system do you used to test KoliBri?
Chrome und Firefox

How to reproduce issue?
Mit eingeschaltetem Screenreader (NVDA) im Fokusmodus (Tab-Taste) durch die Seite navigieren und eine Select-Schaltfläche mit einer Einfachauswahl auswählen. Durch die Auswahlmöglichkeiten (mit der Tastatur) navigieren.

Current vs. Expected:
Aktuelles Verhalten: Alle vier Möglichkeiten der Select-Schaltflächen-Liste werden korrekt vorgelesen. Der Nutzer gelangt jedoch anschließend nicht mehr zu dem Eintrag "no choice" und bleibt bei dem zweiten Listenelement "2 von 4" hängen. Dies kann verwirrend sein. Dies könnte evtl. mit einem aria-hidden-Attribut für die "no choice"-Auswahl korrigiert werden.
Erwartetes Verhalten: Der Nutzer bekommt für das no choice kein "1 von 4" vorgelesen, sodass die Ausgabe für die restlichen korrekt vorgelesen wird. Beispielsweise "1 von 3".
@laske185
Copy link
Contributor

laske185 commented Feb 26, 2025 •
edited
Loading

Das Pattern mit der ersten Option als "Keine Auswahl" oder "Bitte wahlen" ist ein gägngier Standard. Ob diese Option wieder auswählbar ist kommt auf den fachlichen Kontext an. Wenn man etwas auswählen muss, dann wird immer die erste Option deaktiviert, sodass sie zwar initial ausgewählt wird, aber nicht erneut auswählbar wird.

Todo:

  • Sample ändern:
    • Erste Option ist nicht disabled
    • Umbenennen auf "Select salutation"

@laske185 laske185 added v2 v3 Here are issues that need to be resolved for version 3. labels Feb 26, 2025
@laske185 laske185 moved this to 📄 Ready in KoliBri Board Feb 26, 2025
@anicyne anicyne self-assigned this Mar 10, 2025
@anicyne anicyne moved this from 📄 Ready to 🏗 In progress in KoliBri Board Mar 10, 2025
anicyne added a commit that referenced this issue Mar 10, 2025
@anicyne
Enable first option and rename label in select
ebf0846 
Refs: #7226
@anicyne anicyne mentioned this issue Mar 10, 2025
Merged
5 tasks
anicyne added a commit that referenced this issue Mar 10, 2025
@anicyne
Enable first option and rename label in select
88f1d42 
Refs: #7226
@anicyne anicyne mentioned this issue Mar 10, 2025
Merged
5 tasks
This was linked to pull requests Mar 10, 2025
Enable first option and rename label in select #7454
Merged
Enable first option and rename label in select #7453
Merged
@anicyne anicyne moved this from 🏗 In progress to 🔍 Review in KoliBri Board Mar 10, 2025
@deleonio
Copy link
Contributor

Hi @anicyne und @laske185,

no choise sollte weiterhin disabled und nicht auswählbar bleiben. Wir können jedoch unser KolSelect, KolCombobox und KolSingleSelect optimieren, sodass alle disabled-Options aria-hidden="true" sind. Das wäre konsequent, da wir ja bei disabled generell mit der nicht Sichtbarkeit arbeiten.

@deleonio deleonio moved this from 🔍 Review to 🏗 In progress in KoliBri Board Mar 12, 2025
@laske185
Copy link
Contributor

laske185 commented Mar 12, 2025 •
edited
Loading

  • Beispiel mit einmal disabled ("Select salutation") und einmal nicht disabled ("No salutation")
    Image
  • Known Issue anlegen
  • Bei disabed aria-hidden setzen, damit die Gesamtzahl abweicht
    • Kommentar am Code mit Referenz auf Known Issue, da sonst immer wieder Diskussionen kommen.

anicyne added a commit that referenced this issue Mar 12, 2025
@anicyne
Set aria-hidden for disabled options
c6ab5ee 
Refs: #7226
anicyne added a commit that referenced this issue Mar 12, 2025
@anicyne
Fix select test
a891db2 
Refs: #7226
anicyne added a commit that referenced this issue Mar 13, 2025
@anicyne
Set aria-hidden for disabled options
c526a7c 
Refs: #7226
@anicyne anicyne moved this from 🏗 In progress to 🔍 Review in KoliBri Board Mar 13, 2025
@github-project-automation github-project-automation bot moved this from 🔍 Review to ✅ Done in KoliBri Board Mar 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@anicyne anicyne

Labels
v2 v3 Here are issues that need to be resolved for version 3.
Projects
KoliBri Board
Archived in project
Milestone
No milestone
4 participants
@deleonio @laske185 @anicyne @Roman-Use