home-assistant-community-themes / blue-night

Blue Night theme for Home Assistant
https://community.home-assistant.io/t/blue-grey-night-theme/30848
8 stars 4 forks source link

Icon display when not available #37

Closed Romero-KCBS2003 closed 12 months ago

Romero-KCBS2003 commented 12 months ago

I noticed that for entities they are currently unavailable or unknown, the icons are displayed faded bright and for entities they are worked normally and are available, the icons are displayed faded. Shouldn't this be the other way around? BR SmartSelect_20230921_071056_Home Assistant

maartenpaauw commented 12 months ago

I personally don't use Home Assistant anymore and don't have the time to fix this issue, but feel free to submit a pull request with visualizations of the fix and I'm happy to merge it.

sdrapha commented 12 months ago

I would keep the grey shade of the icons though. since changing that would be a massive change to the whole interface which is not welcome. The real issue here, which I agree, is the unavailable entities, they should have the same shade of gray as the normal icons, and not be brighter. If they were to be different, even more faded then. but probably the best would be just the same color. to have a consistent look in the gui.

Romero-KCBS2003 commented 12 months ago

I would keep the grey shade of the icons though. since changing that would be a massive change to the whole interface which is not welcome. The real issue here, which I agree, is the unavailable entities, they should have the same shade of gray as the normal icons, and not be brighter. If they were to be different, even more faded then. but probably the best would be just the same color. to have a consistent look in the gui.

Okay. And do you know how to set in the theme that the icons are always the same shade, especially when they are not available? I'm not familiar with the theme yaml files unfortunately. BR

sdrapha commented 12 months ago

I think I know how to fix that, give me some time and I test the implementation locally and then make a PR.