Autor Thema: fhemSVG-Icons: Fill-Attribute repariert  (Gelesen 426 mal)

Offline Christoph Morrison

  • Full Member
  • ***
  • Beiträge: 130
    • Private Website
fhemSVG-Icons: Fill-Attribute repariert
« am: 12 November 2017, 00:13:21 »
Hallo zusammen,

es hat mich tierisch genervt, dass einige der SVG-Icons nicht (vollständig) umgefärbt wurden, auch wenn man den entsprechenden Farbcode mitgegeben hat. Bei vielen der Icons (106, wenn ich richtig habe zählen lassen) fehlte(n) (das) fill-Attribut(e).

Ich hatte heute Abend noch DS9 fertig zu gucken und hatte deshalb ein wenig Zeit, all diese Icons zu fixen. Ich habe dabei versucht, dort wo keine Umfärbung sinnvoll ist, sie auch nicht zu machen (z.B. das rote Quadrat in rc_RED.svg oder in den Ampeln).

JFTR:
Um die/eine Übersicht zu bekommen, habe ich eine Test-FHEM-Installation mit Vagrant und Puppet gemacht und eine icons.cfg automatisch anhand der Dateien in fhemSVG erstellt, die ich dann in die Vanilla fhem.cfg inkludiert habe. Dabei ist mir lustigerweise aufgefallen, dass man nicht attr ausführen muss, um ein Attribut zu setzen/verändern, sondern es reicht, mindestens at zu schreiben, attr ist nicht nötig. a allerdings rief mal addstruct und mal attr auf.
« Letzte Änderung: 12 November 2017, 00:24:04 von Christoph Morrison »
Projekte:
FHEM-Patches: http://bit.ly/2jivXXu

Offline Wuppi68

  • Developer
  • Hero Member
  • ****
  • Beiträge: 1439
  • Wuppertaler Wimpelbeauftragter
Antw:fhemSVG-Icons: Fill-Attribute repariert
« Antwort #1 am: 12 November 2017, 17:06:05 »
Hallo zusammen,

es hat mich tierisch genervt, dass einige der SVG-Icons nicht (vollständig) umgefärbt wurden, auch wenn man den entsprechenden Farbcode mitgegeben hat. Bei vielen der Icons (106, wenn ich richtig habe zählen lassen) fehlte(n) (das) fill-Attribut(e).

Ich hatte heute Abend noch DS9 fertig zu gucken und hatte deshalb ein wenig Zeit, all diese Icons zu fixen. Ich habe dabei versucht, dort wo keine Umfärbung sinnvoll ist, sie auch nicht zu machen (z.B. das rote Quadrat in rc_RED.svg oder in den Ampeln).

JFTR:
Um die/eine Übersicht zu bekommen, habe ich eine Test-FHEM-Installation mit Vagrant und Puppet gemacht und eine icons.cfg automatisch anhand der Dateien in fhemSVG erstellt, die ich dann in die Vanilla fhem.cfg inkludiert habe. Dabei ist mir lustigerweise aufgefallen, dass man nicht attr ausführen muss, um ein Attribut zu setzen/verändern, sondern es reicht, mindestens at zu schreiben, attr ist nicht nötig. a allerdings rief mal addstruct und mal attr auf.

Wow :-) Sieht auf dem ersten Blick "ziemlich gut aus ....

werde meine SVGs mal patchen um zu sehen, ob es alles klappt und dann mal schauen, wie man die Änderungen am besten in die Updates verteilt

könnte ein wenig zu viel Last auf dem Server geben
Jetzt auf nem I3 und primär Homematic

kein Support für cfg Editierer

Support heißt nicht wenn die Frau zu Ihrem Mann sagt: Geh mal bitte zum Frauenarzt, ich habe Bauchschmerzen

Offline Wuppi68

  • Developer
  • Hero Member
  • ****
  • Beiträge: 1439
  • Wuppertaler Wimpelbeauftragter
Antw:fhemSVG-Icons: Fill-Attribute repariert
« Antwort #2 am: 12 November 2017, 17:14:09 »
Hi,

schaut bei  mir im System gut aus - Danke !!!!

@Rudi: liefern wir diese über 200 Icons ganz normal via Update aus?
Jetzt auf nem I3 und primär Homematic

kein Support für cfg Editierer

Support heißt nicht wenn die Frau zu Ihrem Mann sagt: Geh mal bitte zum Frauenarzt, ich habe Bauchschmerzen

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 17251
Antw:fhemSVG-Icons: Fill-Attribute repariert
« Antwort #3 am: 12 November 2017, 17:35:10 »
Wenn es sein muss, ja. Es sei denn, du hast einen besseren Vorschlag.
Noch ein Argument fuer ein FHEM-Release :)

Offline Wuppi68

  • Developer
  • Hero Member
  • ****
  • Beiträge: 1439
  • Wuppertaler Wimpelbeauftragter
Antw:fhemSVG-Icons: Fill-Attribute repariert
« Antwort #4 am: 12 November 2017, 18:08:57 »
Wenn es sein muss, ja. Es sei denn, du hast einen besseren Vorschlag.
Noch ein Argument fuer ein FHEM-Release :)

nur eine Idee:

den Patch ausliefern und dann starten - geht das?

oder wir machen das ganze Päckchenweise ....

in 4 Pakete aufteilen und dann Rolloutwellen fahren :-)
Jetzt auf nem I3 und primär Homematic

kein Support für cfg Editierer

Support heißt nicht wenn die Frau zu Ihrem Mann sagt: Geh mal bitte zum Frauenarzt, ich habe Bauchschmerzen

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 17251
Antw:fhemSVG-Icons: Fill-Attribute repariert
« Antwort #5 am: 12 November 2017, 18:44:54 »
Zitat
den Patch ausliefern und dann starten - geht das?
Weiss nicht genau, was du damit meinst, aber die Antwort ist vmtl. nein.

Offline Christoph Morrison

  • Full Member
  • ***
  • Beiträge: 130
    • Private Website
Antw:fhemSVG-Icons: Fill-Attribute repariert
« Antwort #6 am: 12 November 2017, 19:05:59 »
Weiss nicht genau, was du damit meinst, aber die Antwort ist vmtl. nein.

Ich habe ja neben den Icons als Files auch einen Patch mitgeliefert, der die SVG-Dateien einfach patcht. Der ist zwar nicht viel kleiner als die Dateien, aber halt nur eine Datei. Diese müsste aber nach dem update noch mit patch (dem Werkzeug) angewendet werden. Ich sehe aber da nicht so den Sinn drin, denn eigentlich ist der Patch für den Maintainer, der den in seinem trunk anwenden soll um die Updates zu bekommen.

Insgesamt bin ich mir aber nicht sicher wovor du Angst hast, @Wuppi68?
Projekte:
FHEM-Patches: http://bit.ly/2jivXXu

Offline Wuppi68

  • Developer
  • Hero Member
  • ****
  • Beiträge: 1439
  • Wuppertaler Wimpelbeauftragter
Antw:fhemSVG-Icons: Fill-Attribute repariert
« Antwort #7 am: 12 November 2017, 20:36:52 »
Ich habe ja neben den Icons als Files auch einen Patch mitgeliefert, der die SVG-Dateien einfach patcht. Der ist zwar nicht viel kleiner als die Dateien, aber halt nur eine Datei. Diese müsste aber nach dem update noch mit patch (dem Werkzeug) angewendet werden. Ich sehe aber da nicht so den Sinn drin, denn eigentlich ist der Patch für den Maintainer, der den in seinem trunk anwenden soll um die Updates zu bekommen.

Insgesamt bin ich mir aber nicht sicher wovor du Angst hast, @Wuppi68?

ich vertraue dem Update Mechanismus von FHEM :-)

jedoch bei der Menge an Dateien steht der Server ein wenig unter Strom - was Traffic und CPU Last bedeutet, und das kann nur Rudi beantworten ;-)
Jetzt auf nem I3 und primär Homematic

kein Support für cfg Editierer

Support heißt nicht wenn die Frau zu Ihrem Mann sagt: Geh mal bitte zum Frauenarzt, ich habe Bauchschmerzen

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 17251
Antw:fhemSVG-Icons: Fill-Attribute repariert
« Antwort #8 am: 13 November 2017, 09:33:21 »
Wenn die Aenderung sinnvoll ist, dann bitte nach dem ueblichen Muster durchfuehren, der Server muss das dann abarbeiten.
Wenn es irgendwo Engpaesse gibt, bitte melden.

Offline Wuppi68

  • Developer
  • Hero Member
  • ****
  • Beiträge: 1439
  • Wuppertaler Wimpelbeauftragter
Antw:fhemSVG-Icons: Fill-Attribute repariert
« Antwort #9 am: 13 November 2017, 22:59:09 »
habe die svgs eingechecked ...

stehen ab sofort auf dem Server via SVN zur Verfügung oder ab morgen via FHEM update

Danke

Ralf
Jetzt auf nem I3 und primär Homematic

kein Support für cfg Editierer

Support heißt nicht wenn die Frau zu Ihrem Mann sagt: Geh mal bitte zum Frauenarzt, ich habe Bauchschmerzen

Offline Wuppi68

  • Developer
  • Hero Member
  • ****
  • Beiträge: 1439
  • Wuppertaler Wimpelbeauftragter
Antw:fhemSVG-Icons: Fill-Attribute repariert
« Antwort #10 am: 14 November 2017, 21:44:51 »
update hat 8 Sekunden gedauert ...

scheint der Server also gut zu packen :-)
Jetzt auf nem I3 und primär Homematic

kein Support für cfg Editierer

Support heißt nicht wenn die Frau zu Ihrem Mann sagt: Geh mal bitte zum Frauenarzt, ich habe Bauchschmerzen

Offline Christoph Morrison

  • Full Member
  • ***
  • Beiträge: 130
    • Private Website
Antw:fhemSVG-Icons: Fill-Attribute repariert
« Antwort #11 am: 15 November 2017, 12:03:09 »
Danke dass ihr die Patches akzeptiert habt. Endlich ist der Raspberry-Böppel bei mir richtig eingefärbt ;-)
Projekte:
FHEM-Patches: http://bit.ly/2jivXXu

Offline Wuppi68

  • Developer
  • Hero Member
  • ****
  • Beiträge: 1439
  • Wuppertaler Wimpelbeauftragter
Antw:fhemSVG-Icons: Fill-Attribute repariert
« Antwort #12 am: 15 November 2017, 14:02:44 »
Danke dass ihr die Patches akzeptiert habt. Endlich ist der Raspberry-Böppel bei mir richtig eingefärbt ;-)

kannst Du die anderen auch noch prüfen/patchen .... oder mir sagen, wie das ganze am einfachsten zu bewerkstelligen ist?

Gruß und Dank

Ralf
Jetzt auf nem I3 und primär Homematic

kein Support für cfg Editierer

Support heißt nicht wenn die Frau zu Ihrem Mann sagt: Geh mal bitte zum Frauenarzt, ich habe Bauchschmerzen

Offline Christoph Morrison

  • Full Member
  • ***
  • Beiträge: 130
    • Private Website
Antw:fhemSVG-Icons: Fill-Attribute repariert
« Antwort #13 am: 15 November 2017, 14:32:57 »
kannst Du die anderen auch noch prüfen/patchen .... oder mir sagen, wie das ganze am einfachsten zu bewerkstelligen ist?

Kann ich machen, ich hab ja nun die Infrastruktur zum Testen.
Projekte:
FHEM-Patches: http://bit.ly/2jivXXu
Gefällt mir Gefällt mir x 1 Liste anzeigen

Offline Christoph Morrison

  • Full Member
  • ***
  • Beiträge: 130
    • Private Website
Antw:fhemSVG-Icons: Fill-Attribute repariert
« Antwort #14 am: 16 November 2017, 08:55:43 »
it_hue_bridge.svg ist mir durchgeschlüpft.

Hier ist
Projekte:
FHEM-Patches: http://bit.ly/2jivXXu