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

Fix unreadable text in light theme and improve accessiblity #37

Merged
merged 3 commits into from
Oct 4, 2024

Conversation

negrel
Copy link
Contributor

@negrel negrel commented Sep 7, 2024

This PR contains 2 commits to fix readability issue and match Grafana accessibility guidelines.

1st commit fix inconsistent text color:

Before After
20240907_15h10m31s_screenshot 20240907_15h39m30s_screenshot

2nd commit fix text color of end of funnel:

Before After
20240907_15h39m30s_screenshot 20240907_15h49m14s_screenshot

This last commit doesn't fix problems introduced by first commit. This was also happening before first commit on light mode.

Let me know if these changes suits you, I'm can make change if needed!

Have a nice day!

@mckn mckn self-requested a review October 4, 2024 09:03
Copy link
Owner

@mckn mckn left a comment

Choose a reason for hiding this comment

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

This is great! Thank you. I will ship it today in a new 1.1.2 release.

@mckn mckn merged commit b3f5585 into mckn:main Oct 4, 2024
8 of 9 checks passed
@mckn
Copy link
Owner

mckn commented Oct 4, 2024

Fixes #40

@szepzoli
Copy link

Hello,

I've just upgraded to 1.1.2, but my my funnel is a bit strange:

image

It might be my mistake, but the lowest value is not displayed properly. By the way: I use another funnel on the same dashboard without any issues.

The one with this strange behaviour has a lots of transformations, but I think it has nothing to do with this bug.

Thank you and Regards,
Zoli

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants