search

ulic75 / power-flow-card

A power distribution card inspired by the official Energy Distribution card for Home Assistant
170 stars 27 forks source link

Solar text color #35

Closed flixlix closed 2 years ago

flixlix commented 2 years ago

I noticed a difference between the "Power Flow Card" and the core "Energy Flow Card". The text of the solar production is black on the original "Energy Card" and orange on the power Flow card. image image

ulic75 commented 2 years ago

In almost all aspects my goal was to mirror the official energy distribution card. However in this case it was an intentional design decision. I feel like it's more consistent with the other bubbles/circles. If you can't stand it I suppose it could be made themable.

flixlix commented 2 years ago

I have thought about this and I don't think I agree with your decision. In the examples of the grid and the battery, the color of the text describes the "throughput" of the line that has that color. For example: In the battery discharging, the green text describes the green line. In the case of solar though, the text being orange, would mean, only the solar power going into the house, which isn't the case, since the solar can also charge the battery or export to the grid.

flixlix commented 2 years ago

For anyone interested, I forked this repo with the same text color as the Energy Flow Card

ulic75 commented 2 years ago

:tada: This issue is resolved in version 2.6.0 :tada:

The release is available on GitHub release