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

đŸ›č Feature: Fehlermeldung in Fieldsets (zusammenhĂ€ngende Eingabefelder) #3852

Closed
gretaneumannn opened this issue Jul 3, 2023 · 6 comments · Fixed by #5655 or #5656
Closed

đŸ›č Feature: Fehlermeldung in Fieldsets (zusammenhĂ€ngende Eingabefelder) #3852

gretaneumannn opened this issue Jul 3, 2023 · 6 comments · Fixed by #5655 or #5656
Labels
desybri The task is assigned to the DESYBRI design system. feature New feature or request

Comments

@gretaneumannn
Copy link

Beispiel fĂŒr zusammenhĂ€ngende Eingabefelder (Fieldsets):

Datum:
image

BetrÀge
image

Anforderungen:

  • Umrandung der fehlerhaften Felder (Vorschlag von Martin mit true und false) ermöglichen
  • Fehlermeldung außerhalb der Input-Komponente muss zugewiesen werden können
  • eine gemeinsame Fehlermeldungen fĂŒr alle Input-Komponente eines Fieldsets

Idee/Orientierung: Gov UK Umsetzung (https://design-system.service.gov.uk/components/error-message/)
@gretaneumannn gretaneumannn added the feature New feature or request label Jul 3, 2023
@deleonio deleonio added the desybri The task is assigned to the DESYBRI design system. label Sep 11, 2023
@deleonio
Copy link
Contributor

Idee:

  • _error=true -> Macht Feld rot ohne Fehlermeldung
    • Problem -> Shadow-Roots zwischen den Inputs
  • Neue Komponente - Fieldset
    • eine Fehlermeldung fĂŒr alle Felder
    • Felder werden ĂŒber JSON definiert
    • Anordnung der Felder flex

Umsetzung nicht trivial.

@deleonio
Copy link
Contributor

Teilaufgaben:

@deleonio deleonio mentioned this issue Sep 18, 2023
Closed
4 tasks
@Chrisdo82
Copy link
Contributor

@deleonio

wenn ich _hideError gesetzt habe und die Fehlermeldung somit ein aria-hidden=true bekommt, wie kann ich dann meine eigene Fehlermeldung mit dem InputFeld verknĂŒpfen? Wenn ich ein aria-describedby ans Eingabefeld packe, dann wird das nicht bis zum input durchgereicht.

This was referenced Nov 23, 2023
Closed
Merged
@deleonio
Copy link
Contributor

@Chrisdo82, @sdvg

Hallo,

vielen Dank fĂŒr den Hinweis. Stefan, wie heute Morgen beschrieben, habe ich versehentlich ein Detail falsch umsetzen lassen. Bitte das aria-hidden wieder entfernen, damit die Fehlermeldung von der KoliBri-Komponente vorgelesen wird und kein aria-discribeedby erforderlich ist.

Christian: Ihr mĂŒsst Eure Ă€ußere Fehlermeldung, die mit der eines Feldes ĂŒbereinstimmen sollte, mit aria-hidden fĂŒr den Screenreader ausblenden.

@Chrisdo82
Copy link
Contributor

Chrisdo82 commented Nov 27, 2023 •
edited
Loading

@deleonio super danke Martin. Wir verwenden fĂŒr unsere Fehlermeldungen auch die KolAlert Komponente. Kann ich der das aria-hidden direkt mitgeben oder muss ich es am Ă€ußeren Wrapper setzen? Und mĂŒsste ich auch das _alert Flag auf false setzen oder ist dass an dieser Stelle irrelevant?

@deleonio
Copy link
Contributor

deleonio commented Nov 27, 2023 •
edited
Loading

@Chrisdo82 gib der kol-alert-Komponente einfach ein aria-hidden="true".

This was linked to pull requests Nov 28, 2023
chore: remove aria hidden by hide error #5655
Merged
chore: remove aria hidden by hide error #5656
Merged
@github-project-automation github-project-automation bot moved this to ✅ Done in KoliBri Board Nov 28, 2023
@Chrisdo82 Chrisdo82 mentioned this issue Dec 12, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
desybri The task is assigned to the DESYBRI design system. feature New feature or request
Projects
KoliBri Board
Archived in project
Milestone
No milestone
3 participants
@deleonio @Chrisdo82 @gretaneumannn