fix for BIESt 1748, closes #1748

assigned to @strohm

marked this merge request as draft

added 1 commit

2ac8897e - fixed text for second navigation

Compare with previous version

added 1 commit

8336b820 - fixed showing the selected element in third navigation for screen readers

Compare with previous version

added 1 commit

1045ad50 - improved reading of second and third navigation layer

Compare with previous version

added 141 commits

1045ad50...bef936d1 - 137 commits from branch main
375081ae - began adding code in navigation for screenreaders
d21e1dde - fixed text for second navigation
ab547e65 - fixed showing the selected element in third navigation for screen readers
4893a93c - improved reading of second and third navigation layer

Compare with previous version

added 1 commit

f59d86c9 - fixed selection text for first level navigation, unified text

Compare with previous version

marked this merge request as ready

Ich frage mich gerade, ob es nicht das einfachste und sinnvollste wäre, wenn aus Navigation::getLinkAttributes() direkt das richtige aria-label rausfällt, wenn das Element aktiviert ist. Dann hätte man nur an einer einzigen Stelle eine Anpassung und die greift immer.

Ich teste das mal.

Das Problem ist, dass die erste, zweite und dritte Navigationsebene durch unterschiedliche Elemente dargestellt werden, sodass der Status für jede Eben anders angegeben werden muss.

Das ist jetzt aber nicht mehr so, oder? D.h. Navigation::getLinkAttributes() könnte das zentral tun. Aktuell setzt die 3. Ebene noch ein aria-label auf den Label, der sowieso anzeigt wird - kann das weg?

Ich habe das mal in !1338 (merged) als ganz einfache Variante so gebaut, das enthält nur drei Änderungen:

aria-current="page" dort entfernen, wo es nicht hingehört
aria-current="page" in Navigation::getLinkAttributes() generieren
nicht verwendeten Code aus templates/tabs.php entfernen - das könnte aber auch ein separates Ticket sein

@strohm: Könntest Du mal probieren, ob das so das richtige tut?

!1338 (merged) ist eine gute Lösung. Ich nehme den.

resolved all threads

Gibt es nicht eigentlich aria-selected (bei Tabs) bzw. aria-current (bei sonstigen Listen), das genau für so eine Markierung gedacht ist?

added 69 commits

f59d86c9...bd6c2bdd - 64 commits from branch main
6fb1f154 - began adding code in navigation for screenreaders
9ee13b48 - fixed text for second navigation
b3ddf34b - fixed showing the selected element in third navigation for screen readers
a0825594 - improved reading of second and third navigation layer
71fe5a04 - fixed selection text for first level navigation, unified text