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

SVG: graph/plot colors #120

Closed mackshot closed 4 years ago

mackshot commented 4 years ago

Colors which are easier do differentiate... Additionally fixed a bug of missing l.fillX-images.

On top the old color-set below the new one color-sets

A preview... preview

klein0r commented 4 years ago

Thanks a lot!

flummy1978 commented 4 years ago

Will jetzt nicht zwingend eine neue issue dafür öffnen (wenn es besser ist sagt bitte bescheid)

Muss man etwas tub, damit die Farben berücksichtigt werden, oder ist es in der aktuellen Version drin? Alternativ (zusätzlich) : Gibt es einen halbwegs einfachen Workarround um die originalen Plot(Farben) und Styles zu nutzen?

Ich finde diesen Style insg mehr als gelungen, ich mag es wirlich sehr damit zu arbeiten. Hier hat Matthias wirklich ganze Arbeit geleistet. Nur die SVG Plots sind grausam.... Dadurch dass ich mobil und Desktop unterschiedliche Styles nutze, hab Ich immer unterschiedliche Farben, wo ich mich dann Immer wieder zurechtfinden muss..... Wäre wkrkli toll :)

klein0r commented 4 years ago

Habe ich gerade in 2.16 gemerged

flummy1978 commented 4 years ago

Wird heute Abend getestet. Dankeschön schonmal 👍

klein0r commented 4 years ago

Dank geht an @mackshot

mackshot commented 4 years ago

Gerne.

Damit der Style hier mehrere Color-Sets unterstützt, müsste man wohl mehr anpassen - ich weiß nicht, ob @klein0r das angehen und supporten will. Ich bin da etwas zwiegespalten. (Der Flex Style hat eine Konfigurationsmaske in der man Einstellungen vornehmen kann. Schick und praktisch für den Anwender, aber erstmal viel Arbeit soetwas zu entwickeln und ob das viele nutzen - man weiß es nicht.)

Um die Farben im Style zu ändern reicht es aus die Datei www/pgm2/hausautomatisierung_comsvg_style.css anzupassen. Ist das ein für Dich gangbarer weg @flummy1978 ? Aber Achtung, wenn die Datei im Rahmen eines Updates verändert wird, verlierst Du Deine Settings wieder.

klein0r commented 4 years ago

Ich hatte mir das mal angeschaut wie die Themes das machen. Am Ende wird einfach nur ein JSON in ein Attribut geschrieben und beim Start wieder gelesen und entsprechend über JS dann CSS generiert. Finde ich... unschön 😄

flummy1978 commented 4 years ago

@mackshot Bevor ich die Benachrichtigung hier bekommen habe, hatte ich mir das in der Zwischenzeit angeschaut und bin auf diese Weise in die Richtung gegangen. Manche Farben waren halt total genial, andere wiederum (für meine persönlichen Zwecke) eher ungeignet. Deshalb ist das eine Zwischenlösung, die ich dann nutze, solange @klein0r keine Updates macht, dann muss ich es wieder nachholen.

@klein0r Warum genau ist "das unschön" ? Es wird ja einen technischen Aspekt geben, der dagegen spricht ? Für mich als User hat dieser Style sehr sehr viele wirklich gute Sachen drin, aber leider eben auch ein paar Nachteile, die für mich persönlich schwer bis unmöglich sind zu korrigieren. Gerade die von @mackshot angebrachten Vorteile, finde ich eben auch sehr gut für den User. Wäre eine alternative nicht Deinen Style um ein "user css" zu erweitern, der dann (ähnlich wie eben im FlexStyle möglich) den betroffnen teil überschreibt ? So wäre der Style ansich immer original (und updatebar) und man könnte Dinge anpassen, wie ich sie gemacht habe: (Hintergrund, gesamtbreite auf 1200 und eben die SVG Plot Farben)

klein0r commented 4 years ago

@flummy1978 User-CSS gibt es doch in FHEM bereits. Siehe Attribute im FHEMWEB. Du kannst das gerne alles programmieren und einen Pull-Request erstellen 😄 Ich habe z.B. nicht ein SVG Plot in FHEM. Entsprechend gering ist meine Motivation da viel Zeit reinzustecken. Immerhin erzählen mir alle ständig, dass es nur eine Admin-Oberfläche und keine Visualisierung ist 😆

Wie gesagt - ich bin offen für Änderungen und Verbesserungen in die Richtung, aber habe zur Zeit viele andere Projekte mit mehr Prio.

mackshot commented 4 years ago

@flummy1978 wenn du willst Bau ich mal einen Prototyp für die Variante mit dem User-css-file. Dauert dann aber ein paar Tage.

flummy1978 commented 4 years ago

@klein0r ... Glaube mir, wenn ich das könnte, hätte ich es schon längt erledigt g Ich bin glaube ich nicht auf den Kopf gefallen, was viele Sachen rund um Fhem / Web etc angeht, aber das ist nicht mehr meine Welt. Dass es direkt in FHEMWEB geht, weiss ich. Ich meinte dass man das im Style Manager bearbeiten kann (wie bei Flex) und es dann eben ins User Style übernommen wird und nicht

über JS dann CSS generiert... und damit "unschön" ist.

Dass Deine Motivation dahingehend gering ist, glaube ich und kann es auch nachvollziehen.

@mackshot Danke für das Angebot, hätte ich nichts gegen zum testen 😃 Grundsätzlich bin ich schon sehr daran interessiert, mögliche Änderungen noch weiter für meine persönlichen Wünsche anzupassen, aber es soll nicht in Arbeit für andere ausarten, damit ich da was von habe. Wenn Du da generell Spaß dran hast, wäre ggf auch eine Zusammenarbeit mit @klein0r (oder zumindest seinem Style, wenn erlaubt), dem Autor vom Flexstyle (der da glaube ich aktuell aus privaten Gründen länger gar nichts mehr macht) . Wenn man es dann im Forum vorträgt, haben weitaus mehr Leute was davon, als nur die die hier lesen. Dort hab ich mich ja auch das ein oder andere Mal im FlexStyle verewigt,bin dann auf meinem Mainsystem doch wieder zu dem HA Style gewechselt - Gewohnheitstier eben 😁

mackshot commented 4 years ago

@flummy1978 Ich muss hier nochmal Klarheit bekommen, was Dein Wunsch ist:

  1. Möchtest Du gerne auf der FHEM Oberflächte über Colorpicker und ähnliches die SVG Farben bearbeiten können, oder
  2. reicht es Dir über Attribute oder zusätzliche CSS Dateien, die du selber bearbeitest die Farben setzen zu können?

Der Ansatz von flex hat ja einige Vorteile, aber eben auch Nachteile, wie z.B., dass man für alles was man verändern möchte auch eine Eingabemaske braucht.