Widget basic command range do nothing

domogik / domoweb

Web interface for Domogik Home Automation free software

24 stars 10 forks source link

Widget basic command range do nothing #54

Closed Basilic closed 7 years ago

Basilic commented 9 years ago

I have add a widget basic range, sensor and command as DT_Scalling.

The send button is disable and the sensor do nothing. If I add a sensor number I can see the sensor working.

If I use the rest url the device working and the sensor value of widget range display the value.

jschaeke commented 8 years ago

+1 for fix

fritz-smh commented 8 years ago

I will check this week

fritz-smh commented 8 years ago

Should be fixed now in develop branch. Please check :)

jschaeke commented 8 years ago

Thanks a lot, I checked it immediately and yes it works! It is however still a little bit awkward as the button does not respond all too well; I have to select it multiple times before I can actually move it. edit: cause of awkwardness: you need to select exact center. Here a video: https://goo.gl/photos/XZsE1xP4otGHSFbE9

jschaeke commented 8 years ago

2 functional issues remain with current version of the slider:

you need to click the exact center of the dot in order to move it (Chrome), in IE the dot is a little square so not applicable
in a mobile browser (Chrome on android/Chrome on IOS), you can move the slider but nothing happens -> an event capture missing?

A third is mere cosmetic one:

After sliding to wanted position, the slider goes back to original position and follows the actual state (my dimmer takes by default 2 seconds to reach end position). It would feel more natural it stays directly at the chosen position (and perhaps see something else reflect the current state: ie: the bar line color creeping up to the button)

fritz-smh commented 8 years ago

should be better now. Can you test ?

jschaeke commented 8 years ago

Just tested on Windows Chrome, Android Chrome and IOS Safari. It works everywhere! Thanks!!