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

Make text more readable in dark mode #10473

Merged
merged 1 commit into from Mar 5, 2024

Conversation

hood
Copy link
Contributor

@hood hood commented Feb 26, 2024

What does this PR do?

This PR enhances the legibility of coloured text in dark mode in some places.

Motivation

Not all coloured labels are easily legible when using the dark mode UI version, significantly impacting its accessibility.

More

  • Added/updated tests
  • Added/updated documentation

Additional Notes

Closes #9623

Preview:

Screen.Recording.2024-02-26.at.13.06.28.mov

@nmengin
Copy link
Contributor

nmengin commented Feb 26, 2024

Hey @hood,

Thanks for the contribution.
With other maintainers, we think that is an accessibility fix.

For such reason, could rebase it on the branch v2.11, please?

@mmatur mmatur added this to the 2.11 milestone Feb 26, 2024
@hood hood force-pushed the more-legible-dark-text branch 2 times, most recently from 4d05802 to 2fb104a Compare March 3, 2024 22:03
@hood hood changed the base branch from master to v2.11 March 3, 2024 22:04
@hood
Copy link
Contributor Author

hood commented Mar 3, 2024

Hey @hood,

Thanks for the contribution. With other maintainers, we think that is an accessibility fix.

For such reason, could rebase it on the branch v2.11, please?

Sure, I’ve just rebased it. Let me know if I can do anything else to help.

@mmatur mmatur added kind/bug/fix a bug fix kind/enhancement a new or improved feature. and removed contributor/waiting-for-corrections kind/bug/fix a bug fix kind/enhancement a new or improved feature. labels Mar 4, 2024
@rtribotte rtribotte changed the title feat: make text more readable in dark mode Make text more readable in dark mode Mar 5, 2024
Copy link
Member

@rtribotte rtribotte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 👍

@nmengin
Copy link
Contributor

nmengin commented Mar 5, 2024

Hey @hood,

Thank you for the contribution 👍

Copy link
Member

@ldez ldez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@traefiker traefiker merged commit 15f5055 into traefik:v2.11 Mar 5, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Color contrast for the dashboard in dark mode is too low
6 participants