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

Active Color no longer set by theme #32

Closed
MrBill12 opened this issue Mar 13, 2023 · 2 comments
Closed

Active Color no longer set by theme #32

MrBill12 opened this issue Mar 13, 2023 · 2 comments

Comments

@MrBill12
Copy link

This might be a duplicate of issue #31 but there were too many colors on that screenshot to understand the issue and expectations. I do not think this is a duplicate of issue #25

I use noctis as installed by HACS, but make two personal edits after each update:

ha-card-border-radius: "35px"
border-radius: "35px"

(including this information since my screenshots will have rounded corners... but otherwise it's noctis)

Since installing 2023.2.2 Active color hasn't worked as expected. (I should note however I actually jumped from 2022.10.7 to 2023.2.2... I had no time in Nov/Dec and then was gone most of January) Specifically for most card types I see Yellow as the active color.

Specifically I would expect front door to display as #1A89F5 blue instead of yellow:
image

Likewise the center "Fireplace" Icon should be #1A89F5 blue instead of yellow: (The YAML shown is the yellow button.)
image

Finally, oddly some add-on cards seem to render as expected, i.e. custom-button card:
image

I think the breakage occurs because of this note in the 2023.2.0 release notes:

Each entity in the frontend has its own color, which is based on the current entity state. We now have officially added support for themes to customize these.

which leads to this section of the front end docs: https://www.home-assistant.io/integrations/frontend/#state-color

I've tried a few dozen experiments to try to find the actual solution, but I guess I don't understand theme's as well as I thought I did.

@aFFekopp
Copy link
Owner

Thanks for your detailed issue report. I have added those new variables in the latest release. Please check if the colors are correct now, I might have missed something.

@MrBill12
Copy link
Author

That seems to have corrected the issue :) Many thanks for fixing it that fast... wish i had told you my guess sooner!

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

No branches or pull requests

2 participants