hotNipi / node-red-contrib-ui-level

A linear level type widget to the Node-RED dashboard
MIT License
27 stars 7 forks source link

Bug with segments color rule #34

Open lucasth24 opened 3 years ago

lucasth24 commented 3 years ago

Describe the bug When trying to set the Min and Max values using the msg.control.seg1/2, i've notice a bug with the color rule. It doesnt respect the limits imposed by the msg.control.seg1/2. After testing, looks like there's some problem related to odd/even numbers, cuz they respond different.

To Reproduce Just enter some msg.control.seg like: 166~175 and 167~176 and move 1 before the lower limit (168) and 1 above the higher limit (177). Expected behavior For exemple, when setted to 166~175 it should be red from 0 to 165, green from 166 to 175, and red from 176 to 250.

Screenshots image

Great work by the way!!

Environment and versions

OS: windows 10 Microsoft Edge Node-RED version: 1.1.3 Dashboard version: 2.24.2 ui-level version: 0.1.40

Additional context Add any other context about the problem here.

hotNipi commented 3 years ago

Would you please create a little test flow for me.

lucasth24 commented 3 years ago

Teste code bellow:

[{"id":"c951ec63.546ad","type":"ui_level","z":"e4085288.0f8a3","group":"b3261d37.ed8d1","order":0,"width":"3","height":"8","name":"1","label":"1","colorHi":"#e60000","colorWarn":"#00ff00","colorNormal":"#e60000","colorOff":"#595959","min":"50","max":"250","segWarn":"","segHigh":"","unit":"","layout":"sv","channelA":"","channelB":"","decimals":0,"animations":"soft","shape":2,"colorschema":"valuedriven","textoptions":"custom","colorText":"#eeeeee","fontLabel":"2","fontValue":"","fontSmall":"","colorFromTheme":true,"textAnimations":false,"hideValue":false,"tickmode":"segments","peakmode":false,"property":"payload","peaktime":3000,"x":290,"y":140,"wires":[]},{"id":"d7d12077.fa5a2","type":"inject","z":"e4085288.0f8a3","name":"seg1","props":[{"p":"control.seg1","v":"166","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":100,"y":80,"wires":[["c951ec63.546ad"]]},{"id":"bd2b9aa5.d94f68","type":"inject","z":"e4085288.0f8a3","name":"seg2","props":[{"p":"control.seg2","v":"175","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":100,"y":140,"wires":[["c951ec63.546ad"]]},{"id":"1bef74e3.10074b","type":"inject","z":"e4085288.0f8a3","name":"","props":[{"p":"payload"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","payload":"176","payloadType":"num","x":100,"y":200,"wires":[["c951ec63.546ad"]]},{"id":"6a3065d6.3c603c","type":"ui_level","z":"e4085288.0f8a3","group":"b3261d37.ed8d1","order":0,"width":"3","height":"8","name":"2","label":"2","colorHi":"#e60000","colorWarn":"#00ff00","colorNormal":"#e60000","colorOff":"#595959","min":"50","max":"250","segWarn":"","segHigh":"","unit":"","layout":"sv","channelA":"","channelB":"","decimals":0,"animations":"soft","shape":2,"colorschema":"valuedriven","textoptions":"custom","colorText":"#eeeeee","fontLabel":"2","fontValue":"","fontSmall":"","colorFromTheme":true,"textAnimations":false,"hideValue":false,"tickmode":"segments","peakmode":false,"property":"payload","peaktime":3000,"x":290,"y":340,"wires":[]},{"id":"757789ab.2c9118","type":"inject","z":"e4085288.0f8a3","name":"seg1","props":[{"p":"control.seg1","v":"166","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":100,"y":280,"wires":[["6a3065d6.3c603c"]]},{"id":"5394ee9a.5285c","type":"inject","z":"e4085288.0f8a3","name":"seg2","props":[{"p":"control.seg2","v":"175","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":100,"y":340,"wires":[["6a3065d6.3c603c"]]},{"id":"a90c7520.34f618","type":"inject","z":"e4085288.0f8a3","name":"","props":[{"p":"payload"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","payload":"166","payloadType":"num","x":100,"y":400,"wires":[["6a3065d6.3c603c"]]},{"id":"8e13dbcd.bed438","type":"ui_level","z":"e4085288.0f8a3","group":"b3261d37.ed8d1","order":0,"width":"3","height":"8","name":"3","label":"3","colorHi":"#e60000","colorWarn":"#00ff00","colorNormal":"#e60000","colorOff":"#595959","min":"50","max":"250","segWarn":"","segHigh":"","unit":"","layout":"sv","channelA":"","channelB":"","decimals":0,"animations":"soft","shape":2,"colorschema":"valuedriven","textoptions":"custom","colorText":"#eeeeee","fontLabel":"2","fontValue":"","fontSmall":"","colorFromTheme":true,"textAnimations":false,"hideValue":false,"tickmode":"segments","peakmode":false,"property":"payload","peaktime":3000,"x":770,"y":140,"wires":[]},{"id":"96e1b14c.b2ea5","type":"inject","z":"e4085288.0f8a3","name":"seg1","props":[{"p":"control.seg1","v":"167","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":580,"y":80,"wires":[["8e13dbcd.bed438"]]},{"id":"d4b67598.b37008","type":"inject","z":"e4085288.0f8a3","name":"seg2","props":[{"p":"control.seg2","v":"176","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":580,"y":140,"wires":[["8e13dbcd.bed438"]]},{"id":"1e1b2cb9.7b77b3","type":"inject","z":"e4085288.0f8a3","name":"","props":[{"p":"payload"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","payload":"168","payloadType":"num","x":580,"y":200,"wires":[["8e13dbcd.bed438"]]},{"id":"891ba684.2d81f8","type":"ui_level","z":"e4085288.0f8a3","group":"b3261d37.ed8d1","order":0,"width":"3","height":"8","name":"4","label":"4","colorHi":"#e60000","colorWarn":"#00ff00","colorNormal":"#e60000","colorOff":"#595959","min":"50","max":"300","segWarn":"","segHigh":"","unit":"","layout":"sv","channelA":"","channelB":"","decimals":0,"animations":"soft","shape":2,"colorschema":"valuedriven","textoptions":"custom","colorText":"#eeeeee","fontLabel":"2","fontValue":"","fontSmall":"","colorFromTheme":true,"textAnimations":false,"hideValue":false,"tickmode":"segments","peakmode":false,"property":"payload","peaktime":3000,"x":770,"y":320,"wires":[]},{"id":"b1a3e70f.c24008","type":"inject","z":"e4085288.0f8a3","name":"seg1","props":[{"p":"control.seg1","v":"167","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":580,"y":260,"wires":[["891ba684.2d81f8"]]},{"id":"b38c6116.b4ed7","type":"inject","z":"e4085288.0f8a3","name":"seg2","props":[{"p":"control.seg2","v":"176","vt":"num"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","x":580,"y":320,"wires":[["891ba684.2d81f8"]]},{"id":"4d82cea0.0d10a","type":"inject","z":"e4085288.0f8a3","name":"","props":[{"p":"payload"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","payload":"177","payloadType":"num","x":580,"y":380,"wires":[["891ba684.2d81f8"]]},{"id":"b3261d37.ed8d1","type":"ui_group","name":"Group 1","tab":"1465caac.846ec5","order":1,"disp":true,"width":"12","collapse":false},{"id":"1465caac.846ec5","type":"ui_tab","name":"Test","icon":"dashboard","order":4,"disabled":false,"hidden":false}]

hotNipi commented 3 years ago

Thank you. Would you please let me know if it works correctly with released version 0.1.41.

lucasth24 commented 3 years ago

Hey! I'm going to stress it and get back to you with some feedbacks!

Thank you very much, amazing job!