aex351 / home-assistant-neerslag-card

Display Buienalarm and/or Buienradar data in a graph for Home Assistant.
29 stars 3 forks source link

Use the colors of the Home Assistant theme and/or user set theme #24

Open aex351 opened 2 years ago

aex351 commented 2 years ago

Colors are currently hardcoded in the Neerslag Card. While the colors do match/resemble the default Home Assistant theme, the colors are not linked. It would be nice if the colors are dynamic and follow the user set theme. This would make it more aligned to the rest of the dashboard and other cards.

Next to this, customizing the colors has also been a request. #4 and https://github.com/aex351/home-assistant-neerslag-app/issues/6

BebeMischa commented 2 years ago

Well, I would love to see this happening too, al I use the dark theme with violet colors, the blue line is now the only one graph with that color...

BebeMischa commented 2 years ago

Coming back to this, It also may be a good idea to have the choice of color for the two sensors.

Like it is now, I can't separate the two lines crossing each other without a mouse-over. On the wall-mounted tablet it's impossible to determine, which line is from which sensor. So two different tones would be really nice ;-)

image

;-)

aex351 commented 2 years ago

Currently the line colors are hard coded and are not following any configured theme colors. I can imagine that for certain user themes the blue color does not match the expected color scheme. I will have a look to see what options are available to use a theme color.

I am unaware of Home Assistant actually having graphs with multiple lines. So I will have a look to see what options are available to use theme colors for multi line graphs. This could be a bit of a challenge.

Second to this all. A fully controlled end-user color configuration (being able to set you own color apart from the theme) would be nice to have.

BebeMischa commented 2 years ago

afbeelding

that's better! thanks ;-)