FHEM Forum

FHEM => Frontends => FHEMWEB => Thema gestartet von: DS_Starter am 18 November 2023, 20:32:05

Titel: Patchvorschlag für variable Feldbreite Widget Text / Multple
Beitrag von: DS_Starter am 18 November 2023, 20:32:05
Hallo Rudi,

bis jetzt wird in der fhemweb.js eine feste Feldbreite 30 für Textfelder (Widget textField...) bzw. Multiple (Widget multiple...) verwendet.

Bei einer meiner aktuellen Applikationen wäre eine einstellbare Feldbreite wünschenswert (Beispiel im Anhang).
Für die Erstellung dieser grafischen Elemente verwende ich die Funktion FW_widgetFallbackFn des FHEMWEB Moduls.

Ich hätte eine Idee/Vorschlag dem User eine optionale Einstellbarkeit der Feldbreiten per FHEMWEB-Attribut anzubieten.
Bist du für eine solche Erweiterung offen?
Wenn ja, würde ich mich hinsetzen und einen Patchvorschlag erarbeiten. Der Patch würde sowohl FHEMWEB als auch fhemweb.js betreffen.

VG
Titel: Aw: Patchvorschlag für variable Feldbreite Widget Text / Multple
Beitrag von: rudolfkoenig am 19 November 2023, 10:55:20
ZitatIch hätte eine Idee/Vorschlag dem User eine optionale Einstellbarkeit der Feldbreiten per FHEMWEB-Attribut anzubieten.
Waere es nicht sinnvoller, die Feldbreite ueber einen weiteren, mit Komma getrennten Parameter in der Widget-Spezifikation einzugeben?

Z.Zt. gilt:
textField[,placeholder]
textFieldNL[,placeholder]
textField-long[,sizePct]
textFieldNL-long[,sizePct]

Ich schlage einen weiteren optionalen Pameter size vor.
Uebersehe ich etwas?
Titel: Aw: Patchvorschlag für variable Feldbreite Widget Text / Multple
Beitrag von: DS_Starter am 19 November 2023, 11:20:11
Naja, hinter dem Change steckt meine Idee, die Einstellung der optimalen Breite dem Anwender zu ermöglichen.
Die optimale Breite hängt von verschiedenen Faktoren wie dem Inhalt der Felder ab.

Deswegen kam mir der Gedanke an Attribute. Mit einem zusätzlichen Parameter könnte ja nur der Modulautor entsprechend agieren. Prinzipiell ja nicht verkehrt, würde aber das ursprüngliche Anliegen nicht wirklich treffen.

Was denkst du?

 
Titel: Aw: Patchvorschlag für variable Feldbreite Widget Text / Multple
Beitrag von: rudolfkoenig am 20 November 2023, 09:31:34
ZitatMit einem zusätzlichen Parameter könnte ja nur der Modulautor entsprechend agieren.
Der Benutzer hat doch die Moeglichkeit ueber das widgetOverride (https://fhem.de/commandref_modular.html#FHEMWEB-attr-widgetOverride) Attribut das bei jedem Geraet unterschiedlich einzustellen.
Duerfte nicht umstaendlicher sein, als ein spezielles Attribut.
Titel: Aw: Patchvorschlag für variable Feldbreite Widget Text / Multple
Beitrag von: DS_Starter am 20 November 2023, 17:08:20
Das widgetOverride Attribut hatte ich völlig verdrängt.  ;)
Ja klar, darüber kann der User sich die Breite dann individuell einstellen.

Unter Berücksichtigung meiner bisher betrachteten Einsatzfälle betrifft es die folgenden Widgets bei denen
das Ergebnistextfeld (eingestellte, gesetzte Werte) einstellbar sein sollte:

textField
textFieldNL
textField-long
textFieldNL-long
multiple
multiple-strict

Das sind auch die, deren Feldbreite zur Zeit fix 30 ist. Time hat die fixe Breite 5. Ich persönlich sehe da keinen Mehrwert dieses Breite änderbar zu gestalten, aber vllt. hat jemand eine andere Meinung dazu oder möglicherweise ist es unter einem ganzheitlichen Gesichtspunkt es sinnvoll auch dieses Widget anpassbar zu gestalten.

Wäre sehr hilfreich wenn du das implementieren würdest.  :)

Grüße,
Heiko
Titel: Aw: Patchvorschlag für variable Feldbreite Widget Text / Multple
Beitrag von: rudolfkoenig am 20 November 2023, 20:02:46
Ich habe das fuer die textField "Familie" eingebaut, und Doku angepasst:
ZitattextField[,placeholder,tfsize] - show an input field. tfsize is the size attribute for the input field, defaults to 30.
textFieldNL[,placeholder,tfsize] - show the input field and hide the label.
textField-long[,sizePct,tfsize] - show an input-field, but upon clicking on the input field open a textArea. sizePct specifies the size of the dialog relative to the screen, in percent. Default is 75
textFieldNL-long[,sizePct,tfsize] - the behaviour is the same as :textField-long, but no label is displayed.

Die NL Varianten habe ich nicht getestet, mangels fertiger Testkonfiguration.

Bei multiple weiss ich nicht, wie/wo ich diesen Parameter angeben soll.
Titel: Aw: Patchvorschlag für variable Feldbreite Widget Text / Multple
Beitrag von: DS_Starter am 20 November 2023, 20:40:44
Ich habe textFieldNL getestet. Funktioniert!

ZitatBei multiple weiss ich nicht, wie/wo ich diesen Parameter angeben soll.
Wäre es eine Variante den letzten Parameter auf ein führendes '#" zu testen? Also z.B.:

multiple,<val1>,<val2>,...,#10
Ist der letzte Parameter ein #... ist es die Angabe der Breite.
Im Sinn der Einheitlichkeit wäre dann allerdings mein Vorschlag die Breite generell mit einem führenden '#' angeben zu lassen auch wenn es bei textField eigentlich nicht nötig ist.
Titel: Aw: Patchvorschlag für variable Feldbreite Widget Text / Multple
Beitrag von: rudolfkoenig am 21 November 2023, 18:31:31
Ich habe dein Vorschlag für multiple implementiert, die textField Familie aber nicht geändert: die Parameter sind mAn unterschiedlich genug.
Titel: Aw: Patchvorschlag für variable Feldbreite Widget Text / Multple
Beitrag von: DS_Starter am 21 November 2023, 20:46:51
Habs gestet ... klappt einwandfrei wie gewünscht.

Vielen Dank Rudi!
Titel: Aw: Patchvorschlag für variable Feldbreite Widget Text / Multple
Beitrag von: DS_Starter am 21 November 2023, 21:01:34
Mir ist gerade aufgefallen dass die neuen Ergänzungen in der deutschen Hilfe zu fhemweb.js noch fehlen!?

LG
Titel: Aw: Patchvorschlag für variable Feldbreite Widget Text / Multple
Beitrag von: rudolfkoenig am 22 November 2023, 17:22:41
Danke, habs nachgezogen.