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

Label zum Suchschlitz hinzufügen (Screenreader-Kompatibilität) #1220

Merged
merged 2 commits into from
May 16, 2024

Conversation

vgerlach
Copy link

Label und title gesetzt; #1219

@j3nsch j3nsch changed the title Update simpleSearchForm.phtml Label zum Suchschlitz hinzufügen (Screenreader-Kompatibilität) May 14, 2024
@j3nsch
Copy link
Member

j3nsch commented May 14, 2024

Das funktioniert so nicht. Wir prüfen die Seite auf valides XHTML, DIV- und INPUT-Element können kein Label-Attribut haben, auch wenn die Browser das tolerieren. In der Fehlermeldung von WAVE geht es um ein Label-Element, das sichtbar sein muss. Das wäre z.B. ein "Suche"-Text vor oder über dem Suchfeld.

Wir könnten auch mit ARIA-Attributen experimentieren, z.B. aria-label anstelle von label.

@j3nsch
Copy link
Member

j3nsch commented May 14, 2024

@stconradr Überlegt Euch mal was hier eine bessere Lösung sein könnte. Die Title-Attribute sind erlaubt, aber hier ist die Frage, ob es sinnvoll ist das ans DIV- und ans INPUT-Element zu hängen. Die Schlüssel search_title und search_symbol_title gibt es nicht. Die ARIA-Attribute können momentan auch nicht validiert werden, aber ich kann dafür sorgen, dass sie ignoriert werden. Die Verwendung eines Label-Elements verändert das Aussehen des Suchschlitzes, wäre aber valides XHTML.

@j3nsch
Copy link
Member

j3nsch commented May 14, 2024

FYI @alw-bsz

@stconradr
Copy link
Contributor

stconradr commented May 14, 2024

Danke für die Info, dann überlegen wir nochmal. Ich persönlich finde die Lösung beim Google-Suchschlitz elegant. Dort ist das Label im Suchschlitz, es wird ein aria-label verwendet.

@j3nsch j3nsch changed the base branch from v4.8.1 to issue1219 May 16, 2024 13:32
@j3nsch j3nsch merged commit 9fbe985 into OPUS4:issue1219 May 16, 2024
0 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants