search

fhem / Timer

FHEM - Module
https://fhem.de/
1 stars 3 forks source link

Create TIMER_Sandro.pm #2

Closed sepultura30 closed 5 years ago

HomeAutoUser commented 5 years ago

Hallo @sepultura30 ich schaue es mir morgen Abend an. Leider muss ich es mir separat vergleichen weil bei dem PR mir alles als Unterschied angezeigt wird und nicht nur die Änderungen von dir. Grüße

sepultura30 commented 5 years ago

Den style war nur für mich und wenn du möchtest gebe ich dir Zeilennummern wo ich was geändert habe.

Von meinem iPhone gesendet

Am 12.10.2019 um 13:23 schrieb HomeAutoUser notifications@github.com:

 @HomeAutoUser commented on this pull request.

In TIMER_Sandro.pm:

@@ -544,7 +539,7 @@ sub Timer_FW_Detail($$$$) {

Überschriften

$html.= "";

  • $style_code1 = "border:1px solid #D8D8D8;" if($Table_Border_Cell eq "on");
  • $style_code1 = "border:1px solid #000000;" if($Table_Border_Cell eq "on"); Entnehme ich dabei den Wunsch, das du nicht die unterschiedlichen Zeilen immer wechselnd gefärbt haben möchtest?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

sepultura30 commented 5 years ago

Das hört sich sehr gut an

Von meinem iPhone gesendet

Am 13.10.2019 um 19:41 schrieb HomeAutoUser notifications@github.com:

 @HomeAutoUser commented on this pull request.

In TIMER_Sandro.pm:

@@ -549,7 +551,7 @@ sub Timer_FW_Detail($$$$) { $html.= "";

for(my $zeile = 0; $zeile < $Timers_Count; $zeile++) {

  • $style_background = "background-color:#000000;" if ($zeile % 2 == 0);
  • $style_background = "background-color:#$Cell_Color;" if ($zeile % 2 == 0); $style_background = "" if ($zeile % 2 != 0); Jetzt muss ich erstmal hier alles auseinader nehmen vom PR ;-) Wir wollen ja ungern nicht verschiedene Dinge vermischen.

Ich würde das mit der Sonnenauf bzw. Untergang in einem Attribut versehen was im Standardfall aus ist und beim setzen greift mit den Usereingaben um die Fälle REAL,CIVIL,NAUTIC,ASTRONOMIC

abzudecken?

StyleColor Hintergrund, hier würde ich im ersten Schritt erstmal den Code so abändern, das der User die Zeilenunterscheidung im jeweilig gewählten Styleformat sieht und wir nicht die Farbe fest legen. Als Option, kann man gern noch ein Attribut ergänzen um eine feste Farbe selbst zu definieren bzw. um die Farbe "Zeilenunterscheidung" abzuschalten.

@elektron-bbs siehst du dies ebenso wie ich? @sepultura30 das müsste doch dann auch so deinem Vorschlag deckeln?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

HomeAutoUser commented 5 years ago

mit https://github.com/fhem/Timer/pull/3 wurden die Anpassungen übernommen