rdmtc / node-red-contrib-ccu

Node-RED Nodes for the Homematic CCU
https://flows.nodered.org/node/node-red-contrib-ccu
MIT License
71 stars 13 forks source link

CCU Switch node - wrong resize in settings dialog #128

Open Sineos opened 3 years ago

Sineos commented 3 years ago

For some reasons the Switch-node's setting dialog resizes wrong. On my screen it thinks that 5062px is about right

grafik

JensUser commented 3 years ago

Bei mir das gleiche. Mit verschiedenen Browsern probiert, überall das gleiche Problem. Bei meiner Vorgängerversion, redmatic-5.5.3, war noch alles in Ordnung.

Michael-Hennemann commented 3 years ago

Update auf Raspberrymatic 3.53.30.20201024 hat nichts geändert bei mir :-/ Edit: Update zu RedMatic 7.0 hat auch nichts gebracht..

Michael-Hennemann commented 3 years ago

@Sineos @JensUser Mal eine blöde Frage: Sind wir die einzigen mit dem Problem? Ich meine schon klar, dass das alles OpenSource und kostenlos ist und wenn ichs könnte würde ich es selbst beheben und ich will mich auch nicht beschweren, aber im Grunde bedeutet dieser "Bug" dass die CCU-Nodes nicht benutzbar sind. Zumindest wenn man in einem Flow den Switch-Node nutzen möchte. Wenn alles das Problem hätten, würde ich meinen, dass das recht weit in der Prio-Liste nach oben rutscht. Im Umkehrschluss: Wenn nicht alle das Problem haben, gibt es vielleicht eine Möglichkeit das selbst zu lösen? Backup -> Neuinstallieren -> Restore? Irgendeine idee?

Sineos commented 3 years ago

Ich selbst bin leider ein ganz lausiger Web-Coder. Hab mir den zugehörigen Code mal angeschaut, aber weder ganz verstanden noch was Auffälliges entdeckt. Meine Vermutung ist, dass ein Update von Node-Red hier jetzt irgendetwas anders handhabt als in der Vergangenheit und es deshalb zu diesem Problem kommt. Finde ich jetzt aber nicht ganz so tragisch, da man sich in Node-Red recht leicht drum herum bauen kann.

Vielleicht findet sich ja jemand mit genug HTML Kenntnisse, das zu korrigieren. Jedenfalls gehe ich davon aus, dass es weder mit Backups oder sonst was zu lösen ist. Maximal mit einem Downgrade auf eine ältere RedMatic Version (vermutlich vor Node-Red 1.1)

Michael-Hennemann commented 3 years ago

Hm, ich würde behaupten es kam mit einem der letzten node-red-contrib-ccu updates. Aber ganz sicher bin ich auch nicht mehr. Hast Du eine Möglichkeit gefunden wie man den Switch-Node trotzdem nutzen kann, (bzw. auf das x klicken kann) oder verwendest Du einfach einen ganz anderen Node?

JensUser commented 3 years ago

Ich nutze einen kleinen Trick. Wenn man aus einem, mit der alten Version erstelltem Flow, den Switch kopiert, dann funktioniert er richtig. Man darf dann allerdings nicht vorher den Switch aus der CCU-Leiste (neu) in den Flow ziehen. Mich wundert allerdings auch , dass es nicht mehr Nutzter sind, die dieses Problem haben.

konne commented 3 years ago

Hi, warum der Trick funktioniert ist mir unklar. Das Problem entsteht hier: https://github.com/rdmtc/node-red-contrib-ccu/blob/master/nodes/ccu-switch.html#L268

Als kleinen Workaround könnte man dort:

let newWidth = rule.width();
if (newWidth > 500) newWidth = 500

schreiben, auch wenn es das Problem nicht wirklich löst, da dann die elemente in der Liste nicht mehr breiter als 500px werden.

Michael-Hennemann commented 3 years ago

@konne Ich hab keine Ahnung wie NodeRed unter der Haube funktioniert, aber kann es vielleicht sein, dass neue Nodes aus einer art Vorlage heraus erstellt werden, Bestehende aber bei einem Updates nicht auf eine neue Version migriert/verändert werden, um die Funktionsfähigkeit nicht zu gefährden? Das würde erklären warum bestehende und Kopien von bestehenden Nodes funktionieren, aber die, die man neu aus der Leiste zieht kaputt sind.

Das würde dann ja auch bedeuten, dass der Bug im node-red-contrib-ccu steckt und mit einem der letzten Updates gekommen ist.

Liegt die Datei da im Klartext auf meiner CCU/Redmatic herum? Kann man die da direkt per SSH editieren? Vielleicht kriegen wir ja hier einen kleinen Workaround-Guide hin bis sich jemand der sich wirklich damit auskennt das Problem richtig beheben kann :-)

konne commented 3 years ago

@Michael-Hennemann ja die Datei liegt genauso bei dir in deiner Node-Red installation und du kannst sie einfach editieren.

@hobbyquaker kannst du dich der sache bitte annehmen, danke

Michael-Hennemann commented 3 years ago

@konne Hm, wo würde ich denn die Zeilen einfügen. 😬 Meine ersten beiden Versuche haben dazu geführt, dass alle CCU nodes weg waren. Zum Glück hatte ich ein Backup der Datei gemacht 😅😅

Michael-Hennemann commented 3 years ago

Habe grade beide Raspberrymatics auf 3.57.4.20210320 gebracht und auch Redmatic auf 7.2.1 aktualisiert und der Fehler tritt bei mir nicht mehr auf. Soweit ich das beurteilen kann ist das Problem behoben. Vielen Dank!!!! 😃😃😃