-
Notifications
You must be signed in to change notification settings - Fork 65
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
Forecast bars contrast is too low in light mode #142
Comments
thanks for reporting the issue. you have a very valid point. i never used the card with white background. will have a look on how to fix the issue shortly. |
Hey @pkissling, can I asked what are the colour hex values and corresponding temps you’ve used? I’d like to match the rest of my theme to them. I’d suggest a new feature to change the colours aka mini-graph-card, but I like them as they are. |
hi there, sorry that i haven't found the time yet to work on this issue. |
I worked around the issue for myself by removing the background and adding a bunch of CSS filters to improve contrast and saturation using card_mod:
style: |
.card-content {
padding: 0 8px !important;
}
forecast-temperature-bar-background {
display: none;
}
forecast-temperature-bar {
filter: brightness(80%) saturate(200%) contrast(200%);
}
forecast-icon {
filter: saturate(200%) contrast(200%) brightness(60%);
} |
Great idea with card_mod. I've added one entry so that the main icon for gets also updated.
|
Checklist:
Release with the issue:
1.0.10
Last working release (if known):
n.a.
Browser and Operating System:
n.a.
Description of problem:
The contrast between the forecast bar foreground and background is too low when the background is white. Ideally I'd like to have no background at all, since I'm trying to use the card on an e-Ink display.
Javascript errors shown in the web inspector (if applicable):
n.a
Additional information:
I have deficient color vision so if it looks fine to you, consider making the card more accessible to colorblind people anyways
The text was updated successfully, but these errors were encountered: