2.17 zerfleddert Content Bereich (rechte Seite) u.a.

klein0r / fhem-style-haus-automatisierung

Custom style for your FHEM project - look and feel like haus-automatisierung.com

https://haus-automatisierung.com/

MIT License

44 stars 25 forks source link

2.17 zerfleddert Content Bereich (rechte Seite) u.a. #146

Closed reibuehl closed 3 years ago

reibuehl commented 3 years ago

Ich hab gerade die 2.17 ausprobiert und bin geschockt vom ganzen aussehen. Bei Räumen mit mehreren Gruppen (attr group) werden die Gruppen mit unterschiedlicher Breite angezeigt, was die gesamte Darstellung zerfleddert. Außerdem wird die Eingabezeile in Seitenbreite ganz am oberen Rand noch über dem Logo angezeigt. Ist das ein Bug oder ein Feature? Ich hab nach 2 Minuten einen Restore auf die vorherige Version gemacht....

klein0r commented 3 years ago

Ich habe einen Pull Request von @ole1986 angenommen - das ist das Ergebnis. 😭

reibuehl commented 3 years ago

Also wenn es geht, solltest Du den wieder raus machen... das Ergebnis ist echt eine Katastrophe.

ole1986 commented 3 years ago

Master branch, nicht den pr

meier81 commented 3 years ago

Also ich muss leider auch sagen das ich heute morgen das Update mit lauter Vorfreude installiert habe, sind ja doch immer wieder Verbesserungen drin. Von daher erstmal vielen Dank für die Arbeit die du hier reinsteckst. Als ich dann aber den Browser aktualisiert hatte muss ich sagen bin ich auch erschrocken, das aussehen des Menüs hat sich etwas geändert, ist aber soweit okay. Wie mein Vorredner schon sagte ist die Eingabezeile jetzt ganz oben über die gesamte Breite, gefällt mir nicht so wirklich ist aber Geschmackssache. Was mich wirklich stört ist das Logfenster fängt jetzt erst unterhalb des Menüs an und nicht wie vorher oben am Seitenanfang, zudem sind alle Gruppen unterschiedlich breit. Im Raum sind diese zwar gleich breit, ich habe jetzt aber Einträge wie den Spritpreis, da ist das Feld gerade noch 2cm breit. Finde ich nicht wirklich gut. Vielleicht sollte man überlegen ob manche Sachen wieder zurückgenommen werden.

ole1986 commented 3 years ago

@meier81 screenshots?

Das mit der Logfile hängt leider bisher vom inhalt ab. Sofern dieser absolute kein leerzeichen beinhaltet bricht der inhalt nach unten. Ich werde das prüfen

ReviloEgros commented 3 years ago

Ich hatte den Fork zum responsive design aus dem Forum auch schon ausprobiert und gleich wieder gelöscht. Sagt mir so gar nicht zu. Schade. Also auch erstmal wieder ein restore machen.

meier81 commented 3 years ago

Bin auch wieder zurück zur 2.16, ist so leider nichts für mich.

meier81 commented 3 years ago

Hier noch ein paar Screenshots von der 2.16 (alt) und der neuen 2.17 (neu). Ich glaube man kann die Unterschiede bzw. Probleme gut erkennen.

Logfile alt:

Logfile neu:

Einträge im Raum alt: Raumeinträge alt

Einträge im Raum neu: Raumeinträge neu

Ansicht des Save Konfig-Buttons alt: Ansicht ungespeicherte Änderung alt

Ansicht des Save Konfig-Buttons neu: Ansicht ungespeicherte Änderung neu

Und noch ein Bild vom neuen Menü und der Eingabeleiste ganz oben: Aussehen neu

ole1986 commented 3 years ago

@meier81 vielen dank für die konstruktive Kritik

Könntest du noch einen Screenshot posten, mit devices die ein oder mehrere funktionen aufweisen? Z.b. ROLLO (Runter/hoch/...) oder Lichtschalter, etc...

Vielen Dank im voraus

klein0r commented 3 years ago

Die Frage ist, wie wir hier weiter machen. Sonst muss ich eine 2.18 bauen, welche identisch zu 2.16 ist. Den Style haben hunderte Leute installiert und überall geht es gerade kaputt 😢

reibuehl commented 3 years ago

Sonst muss ich eine 2.18 bauen, welche identisch zu 2.16 ist.

Ja, bitte!

PureSecDE commented 3 years ago

Fehler passieren, kein Problem.

Aber eine doofe Frage, wird so etwas vor einem neuen Release nicht getestet und die korrekte Anzeige geprüft? Oder liegt es evtl. am Browser und mit dem "richtigen" Browser wird es korrekt angezeigt?

Ich habe den Edge und den Firefox geprüft, beides unschön, wie in den Screenshots von meier81 zu sehen.

klein0r commented 3 years ago

Habe 2.18 freigegeben.

@PureSecDE Ich selbst stecke hier keine Entwicklungszeit mehr rein. Deswegen war das etwas halbherzig getestet à la wird schon gut gehen. Zumal in der Vergangenheit immer gute Änderungen eingereicht wurden. Ist dieses Mal halt schief gegangen und jetzt wieder in Ordnung.

reibuehl commented 3 years ago

Danke!!!

ole1986 commented 3 years ago

Da sich @klein0r von diesem Projekt distanziert und unbekannt ist wann es ein nächstes update gibt. Würde ich euch bitten die aktuelle version von mir über den nachfolgenden Weg zu testen.

Version 2.2.0

update add https://raw.githubusercontent.com/ole1986/fhem-style-haus-automatisierung/master/controls_ha_theme_ole1986.txt
update check ha_theme_ole1986
update all ha_theme_ole1986

Somit kann auch nun schneller zwischen dem Original und meinen Anpassungen gewechselt werden.

Zurück zum Original

update check ha_theme
update all ha_theme

klein0r commented 3 years ago

@ole1986 Nur das control file zu trennen ist keine so gute Idee - dann geht bei einem update all alles durcheinander.