feat: Essential icon: show solar colour from top down

RhysMcW commented 1 month ago

Description

Amend the linearGradient calculations to show the solar colour from the top down

Related issues

closes #428

Motivation and Context

Solar is from above, so show the solar colour frmo the top down.

How has this been tested

built a copy of sunsynk-power-flow-card.js and installed it on my HA box and checked display of essential icon.

Type of change

[ ] New feature (non-breaking change which adds functionality)

Checklist

[x ] My code follows the style guidelines of this project
[x ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules
[x ] I have updated the version in package.json following semver

slipx06 commented 1 month ago

@RhysMcW I've made some changes to the logic ~~on the compact card~~. Some of the checks were redundant. Can you please test.

RhysMcW commented 1 month ago

@slipx06, thanks, have built a new dist file and applied it to my HA - will have to wait until the morning, when I get PV again, to check

RhysMcW commented 1 month ago

looks good this morning ess icon - pv from top 3

slipx06 commented 1 month ago

It's not working properly for me but I think I see what the issue is

slipx06 commented 1 month ago

@RhysMcW thanks for the help with this. I've decided to simplify the logic and implement this slightly differently. Closing this pull request. This will be added to v4.37.0

slipx06 / sunsynk-power-flow-card