-
-
Notifications
You must be signed in to change notification settings - Fork 331
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
[Progress] Display dark text on light inverted backgrounds #196
[Progress] Display dark text on light inverted backgrounds #196
Conversation
Matter of taste 😄 |
I agree with @ColinFrick so that it is consistent. |
OK, convinced 😉. I am gonna changing this Pr accordingly and for toast I am creating a separate PR |
Was also on @ColinFrick's opinion. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Some progress text is unreadable on inverted progress bars, because their background color is too light.
This PR changes the text colors from white to black on
inverted yellow|olive|teal|grey progress
Screenshot
Before
After