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

🐞 Bug: 9.1.4.3 Badge: Textkontrast ist zu gering #5519

Closed
fohi17 opened this issue Nov 3, 2023 · 4 comments · Fixed by #5540 or #5547
Closed

🐞 Bug: 9.1.4.3 Badge: Textkontrast ist zu gering #5519

fohi17 opened this issue Nov 3, 2023 · 4 comments · Fixed by #5540 or #5547
Assignees
Labels
a11y-retest bug Something isn't working

Comments

@fohi17
Copy link

fohi17 commented Nov 3, 2023

Note

  • Ich achte beim Erstellen des Issues darauf, dass ich keine fachlichen und unternehmensinternen Details preisgebe.

Textkontrast von Badges ist zu gering

Beschreibung des Fehlers

Der Textkontrast von blauen Badges (4,1:1) und grünen Badges (4,4:1) ist unter 4,5:1 und ist daher nicht ausreichend.

Hinweis: Der Kontrast des roten Badge ist 6,7:1 und daher unter der AAA-Grenze.

Reproduktion

Style: BMF, Z2
Komponente: badge basic (https://public-ui.github.io/sample-react/#/badge/basic)
Version: v2.0.0-rc.6

Schritte zum Reproduzieren des Verhaltens:

  • Kontrastmessung mit dem Color Contrast Analyser

Erwartetes Verhalten

Der Kontrast muss mindestens über 4,5:1 sein (im Idealfall über 7:1).

@fohi17 fohi17 added the bug Something isn't working label Nov 3, 2023
@deleonio
Copy link
Contributor

deleonio commented Nov 3, 2023

Die Badges sollten alle min. 7:1 haben. Dafür ist ein Algorithmus zuständig. Er berücksichtigt jedoch nicht die Schrifdicke.

Bitte nachlesen.

@deleonio
Copy link
Contributor

deleonio commented Nov 6, 2023

image

Die Dev-Tools bestätigen den Fehler.

Bitte für unserer Beispielfarben mit ausreichend Kontrastabstand wählen. Farben aus dem mittleren Spektrum können weder zu schwarz noch zu weiß genügend Kontrast haben.

@sdvg sdvg self-assigned this Nov 7, 2023
@sdvg sdvg linked a pull request Nov 7, 2023 that will close this issue
@deleonio deleonio linked a pull request Nov 9, 2023 that will close this issue
@fohi17
Copy link
Author

fohi17 commented Nov 24, 2023

Huhu @deleonio & @sdvg. Kann man das wieder auf machen oder noch korrigieren? Die Farben in der "badge basic" Komponente haben jetzt einen Kontrast von 7:1, bei "badge button" sehe ich aber noch die alten Farben : )

@sdvg
Copy link
Member

sdvg commented Nov 24, 2023

Danke @fohi17! Ich habe hier ein Followup-Ticket angelegt und werde mich zeitnah darum kümmern: #5622

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y-retest bug Something isn't working
Projects
Archived in project
5 participants