RaspberryMatic - Verbesserungsvorschläge/Wünsche

[Meister]

Guten Morgen zusammen

Habe es mal ausprobiert, leider verändert es die Art der Sortierung nicht.
Aber Danke für die Idee!

[MichaelN]

Dann hast du nicht richtig getestet, denn 0 ist kleiner als 1 - auch als string und daher muss 01 vor 10 einsortiert werden.

[jp112sdl]

Interessant, wenn man mal einen Abend hier nicht mitliest.

Da versuchen 4-5 Leute einen Konsens bzgl. der Design-Anpassungen zu finden und dann kommen hier plötzlich Einwürfe aus'm Gebüsch...
Ich hoffe, das mit der Sortierung ist mittlerweile gelöst? Zeit wird's allemal.

Tut mir auch wirklich leid, dass ich wieder zum vorherigen Inhalt zurück komme:

Könnte man dann auch bitte den auf der Startseite "angepinnten" Systemvariablen wieder etwas mehr Raum gönnen?

Gerne, aber auch da hab ich leider noch keine CSS Methode gefunden ob die Breite der ersten Spalte der Tabelle mit den Systemvariablennamen immer automatisch so breit zu machen wie der längste Text.

Wie war denn das bisher? Da war das doch ganz passend.
Eine Tabelle mit 2 Spalten und 40%/60% Aufteilung.
Schön dynamisch, der Name wird bei schmaler werdendem Fenster dann irgendwann automatisch umgebrochen.

Jetzt sind aber fest 170px eingestellt?
Ich seh grad den "Benefit" deiner Style-Anpassung an dieser Stelle grad noch nicht.

[Baxxy]

Ich seh grad den "Benefit" deiner Style-Anpassung an dieser Stelle grad noch nicht.

Wir müssen erstmal auf den heutigen Snapshot warten, da hat @jmaus in diesem Bereich was angepasst.
(oder hast du manuell gepatcht?)

Grüße, Baxxy

[jp112sdl]

Wir müssen erstmal auf den heutigen Snapshot warten,

Jetzt wo du es sagst... Der war doch sonst schon morgens um 6 verfügbar !?

[jmaus]

Wir müssen erstmal auf den heutigen Snapshot warten,

Jetzt wo du es sagst... Der war doch sonst schon morgens um 6 verfügbar !?

Die Buildserver haben gerade anscheinend ein Problem. Schaue ich mir nachher an und liefere die snapshots dann nach.

[jmaus]

Trotzdem ist das schon irgendwie ein valider Punkt. Die Sortierung ist ja einfach durch Stringvergleich in iseSort / iseSortMultiArray gelöst. Man kann natürlich auch den sogenannten natural / human sort anwenden. Es gibt da auch Lösungen in JS ... aber das wird ein Spaß in der WebUI anzupassen und zu testen.

Da geb ich dir recht, das das prinzipiell natürlich ein valider Punkt ist und man kann drüber nachdenken die Sortierungsroutinen anzupassen. Und wenn es da schon fertige Lösungen in JS gibt, dann her damit! Warum muss nur meistens ich immer solche Dinge anpassen wenn in der Community weit mehr Web/JS-Erfahrung existiert?

[ptweety]

... man kann drüber nachdenken die Sortierungsroutinen anzupassen. Und wenn es da schon fertige Lösungen in JS gibt, dann her damit!

Dann nur mal für spätere Referenz hier dieser Codescchnipsel:

Code: Alles auswählen

console.log(['Z', 'a', 'z', 'ä', '0', '2', '10', '1a', 'a0', 'a1', 'a10', 'a2'].sort(new Intl.Collator(undefined, {
  numeric: true,
  sensitivity: 'base'
}).compare));

und

Code: Alles auswählen

const collator = new Intl.Collator(undefined, {numeric: true, sensitivity: 'base'});
console.log([{'name': 'Z', 'room': 'A'}, {'name': 'a', 'room': 'A'}, {'name': 'z', 'room': 'A'}, {'name': 'ä', 'room': 'A'},
             {'name': '0', 'room': 'B'}, {'name': '2', 'room': 'B'}, {'name': '10', 'room': 'B'}, {'name': '1a', 'room': 'B'},
             {'name': 'a0', 'room': 'C'}, {'name': 'a1', 'room': 'C'}, {'name': 'a10', 'room': 'C'}, {'name': 'a2', 'room': 'C'}]
              .sort( (x, y) => { return collator.compare(x.name, y.name); } ) );

Warum muss nur meistens ich immer solche Dinge anpassen wenn in der Community weit mehr Web/JS-Erfahrung existiert?

Keine Ahnung. Liegt vielleicht daran, dass das Coding in WebUI.js so abschreckend ist?! Das sieht alles so aus, als hätten die Praktikanten der letzten 15 Jahre alle mal rangedurft.

[stan23]

Danke für eure Bemühungen. Ich lese bei euren Debug-Erkenntnissen gerne mit

Eine Sache ist mir gerade aufgefallen:

/www/webui/js/lang/de/translate.lang.extension.js, Zeile 116:

"dialogCreateLinkErrorContent6" : "<u>Achtung</u><br/> Es besteht die M%FCglichkeit, dass die maximale Anzahl der erlaubten Verkn%FCpfungen erreicht wurde.",

Es heißt natürlich nicht Müglichkeit

Richtig wäre:

"dialogCreateLinkErrorContent6" : "<u>Achtung</u><br/> Es besteht die M%F6glichkeit, dass die maximale Anzahl der erlaubten Verkn%FCpfungen erreicht wurde.",

Wäre schön wenn das bei Gelegenheit mit gefixt wird.
Ansonsten mache ich gerne einen PR oder Patch, wenn mir ein kurzer Hinweis gegeben wird wogegen.

EDIT:
https://github.com/jens-maus/RaspberryMatic/pull/1480

[jmaus]

Warum muss nur meistens ich immer solche Dinge anpassen wenn in der Community weit mehr Web/JS-Erfahrung existiert?

Keine Ahnung. Liegt vielleicht daran, dass das Coding in WebUI.js so abschreckend ist?! Das sieht alles so aus, als hätten die Praktikanten der letzten 15 Jahre alle mal rangedurft.

Das mag sein, aber es ist halt wie es ist und solange wir keine komplett eigene/neue WebUI haben müssen wir mit dem leben was wir haben und ggf. das dann eben hier+da noch umpatchen. Wenn du also da deine expertise einbringen kannst, dann gerne! Ich weiss das ist viel try&error und fremden code anpassen und dann noch mit der premisse nicht alles gänzlich umzustellen/neuzuschreiben ist nicht so schön, aber es ist eben ein Anfang!