Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Panel: Low text contrast for selected dropdown items #6664

Closed
nilshoerrmann opened this issue Sep 11, 2024 · 0 comments · Fixed by #6672
Closed

Panel: Low text contrast for selected dropdown items #6664

nilshoerrmann opened this issue Sep 11, 2024 · 0 comments · Fixed by #6672
Assignees
@distantnative
Labels
type: a11y 👐 Concerns a change that would make it more accessible to more people
Milestone
4.5.0

Comments

@nilshoerrmann
Copy link
Contributor

Description

When using a dropdown with light theme, the text contrast for active items is not sufficient ("Dieser Monat" in this case):

grafik

grafik

Expected behavior
The text contrast should be sufficient to read. This text should actually be the most prominent one in the list, so maybe going with a bolder weight might be better than a different colour.

Your setup

Kirby Version
Latest version.
@distantnative distantnative added the type: a11y 👐 Concerns a change that would make it more accessible to more people label Sep 11, 2024
@distantnative distantnative linked a pull request Sep 14, 2024 that will close this issue
Better contrast for current item in light dropdown #6672
Merged
@distantnative distantnative self-assigned this Sep 14, 2024
@distantnative distantnative added this to the 4.5.0 milestone Sep 23, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@distantnative distantnative

Labels
type: a11y 👐 Concerns a change that would make it more accessible to more people
Projects
None yet
Milestone
4.5.0
Development

Successfully merging a pull request may close this issue.

Better contrast for current item in light dropdown getkirby/kirby
2 participants
@nilshoerrmann @distantnative