search

artem-sedykh / mini-climate-card

Minimalistic climate card for Home Assistant Lovelace UI
MIT License
262 stars 19 forks source link

Hide Indicator PR #115

Closed adi90x closed 1 year ago

adi90x commented 1 year ago

Hello,

As discussed in #114 , a quick PR to be able to hide an indicator based on a function result. ==> Give a way to display indicator only in certains circonstances.

Regards, Adrien

regevbr commented 1 year ago

Thanks for the PR! just a small change and request to implement this feature across other hide configs

regevbr commented 1 year ago

@adi90x if you want, I can take this PR and complete it myself as I want to take this chance to make this feature bigger and apply it to all components.

adi90x commented 1 year ago

Yes no problem it is all yours 😀

Le sam. 8 avr. 2023, 09:30, Regev Brody @.***> a écrit :

@adi90x https://github.com/adi90x if you want, I can take this PR and complete it myself as I want to take this chance to make this feature bigger and apply it to all components.

— Reply to this email directly, view it on GitHub https://github.com/artem-sedykh/mini-climate-card/pull/115#issuecomment-1500814762, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACVGU5HP4I475MPPCENWQN3XAEH2NANCNFSM6AAAAAAWULEAXI . You are receiving this because you were mentioned.Message ID: @.***>

regevbr commented 1 year ago

done in #118 and released in 2.5.0. @adi90x would appreciate if you can verify that it works well - specifically that it reacts dynamically