Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Änderung Update-Mechanismus bei Einzeilig #15

Open
wants to merge 1 commit into
base: fix_UI
Choose a base branch
from

Conversation

bismosa
Copy link
Collaborator

@bismosa bismosa commented Aug 30, 2019

Siehe #11

@HomeAutoUser
Copy link
Collaborator

Bitte aufpassen, auch hier #16 das der Grundcode https://github.com/HomeAutoUser/SD_Keeloq__old_Jaro sich geändert hat nach kleinen Fixes und Behebungen. Wir kommen sonst auf Konflikte zu ;-)

@HomeAutoUser
Copy link
Collaborator

@bismosa
wenn du wieder die Zeit findest, so würde ich dich mal bitten dies https://github.com/HomeAutoUser/SD_Keeloq__old_Jaro/tree/fix_UI zu testen. Ich habe mal herumgespielt und vergleichen. Mit dem sehr geringeren TimeOut, so wird das wahrnehmbare flackern fast unsichtbar hier.

Ich habe das System auch "nur" auf einem Pi und wir sollten ja immer auch nutzen zum Zweck sehen.
Ja, wir suchen noch den Weg um es anders zu updaten aber wieso sollten wir den Code noch mehr größer sprengen wenn man es kurz haben kann. Dagegen sollten wir auch schauen, wieviel User nutzen das UI Einzeilig? Stets aber auch im Kopf haben, das das Modul selbst wachsen kann wenn neue Geräte hinzukommt, da es ja schon gerade nicht nur für Jaro geschrieben ist.

Alles Punkte welche wir abwägen sollten.

PS: Eine Diskussion ist keine Kritik :-)

@bismosa
Copy link
Collaborator Author

bismosa commented Sep 1, 2019

Hallo!
Eigentlich könnte es mir ja egal sein...ich werde die einzeilige Variante vermutlich nie selbst nutzen.
Ich denke aber das auch diese Variante gerne benutzt werden wird...deswegen ist mir das dann doch nicht so egal :)
Wenn es als einziges Device im Raum ist, ist der Reload bei mir ebenfalls schnell.
Wenn ich das im gleichen Raum habe wie auch meine anderen 4 Fernbedienungen (mit mehrzeilig) ist der Reload schon deutlich spürbar.
Wenn ich dann noch mein SVG im Raum habe ist der Reload mit über 3sek. nervig.

Ich hatte da auch noch eine Idee, die ich jetzt mal ausprobiert habe. Mal als Diskussionsgrundlage.
#18

Hier wird jetzt bei der Änderung der Auswahl nur noch der Wert im Reading gemerkt, aber keine Weitere Aktion ausgeführt.
Wenn man einen Button betätigt, wird per JavaScript der FHEM Befehl abgesetzt...dabei wird vorher der Kanal aus dem Optionsfeld hinzugefügt.
Damit entfällt der Reload. Bei einem SeitenReload wird der zuletzt eingestellte Kanal wieder übernommen...und in anderen Fenstern bleibt es bei der Einstellung, die auch zumindest in meinem Test problemlos funktionierte.

Lässt sich bestimmt noch verbessern. Kann ich Anfang der Woche mal probieren. Ist von Unterwegs nicht so einfach :)

Gruß
Bismosa

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants