Closed Leithian closed 9 months ago
It's probably all ok, but let me check also tomorrow.
It seems odd to have no default options configured, but always creating the same property in the constructor.
Review comments for FanChip.js
, also apply to the other chip classes.
The switch chip is, in my case, a dangerous chip.. When clicking on it, it triggers all the switches so that the fuses blow!
BTW: the fuses don't blow. One of the switches that turned off was a sort of main switch in my fuse box. My bad :-)
Resolved with chips: switch_count: false Now the chip is not shown!
The switch chip is, in my case, a dangerous chip.. When clicking on it, it triggers all the switches so that the fuses blow!
This PR is pretty much stale... What do you suggest for improving the chips? Please discuss it in the discussions section or create an issue.
Closed for being stale.
I found it confusing that some chips directly navigate when tapping/clicking while others will perform an action. This has caused me to turn off all my lights/switches multiple times, causing some problems.
I've added an option to swap the tap and hold actions where applicable.
If I'm the only one with this problem feel free to close this PR, I can keep my own private version.