Create TIMER_Sandro.pm #2
Conversation
|
Hallo @sepultura30 |
TIMER_Sandro.pm
Outdated
| @@ -544,7 +539,7 @@ sub Timer_FW_Detail($$$$) { | |||
| ## Überschriften | |||
| $html.= "<tr>"; | |||
| #### | |||
| $style_code1 = "border:1px solid #D8D8D8;" if($Table_Border_Cell eq "on"); | |||
| $style_code1 = "border:1px solid #000000;" if($Table_Border_Cell eq "on"); | |||
There was a problem hiding this comment.
Entnehme ich dabei den Wunsch, das du nicht die unterschiedlichen Zeilen immer wechselnd gefärbt haben möchtest?
There was a problem hiding this comment.
Hi,
habe noch mal was am Code geändert, man kann per Attribut jetzt die Zellen Farbe wählen.
|
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 ***@***.***>:
@HomeAutoUser commented on this pull request.
In TIMER_Sandro.pm:
> @@ -544,7 +539,7 @@ sub Timer_FW_Detail($$$$) {
## Überschriften
$html.= "<tr>";
####
- $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.
|
| my $Sunrise = AttrVal($name,"Sunrise","REAL"); | ||
| my $Sunset = AttrVal($name,"Sunset","REAL"); | ||
| my $Sunrise = AttrVal($name,"Sunrise",""); | ||
| my $Sunset = AttrVal($name,"Sunset",""); |
There was a problem hiding this comment.
Benötigst du wirklich zwei Attribute für Sonnenafgang und Sonnenuntergang, oder reicht auch eines für beide Zeiten?
There was a problem hiding this comment.
Es reicht auch eines für beide Zeiten
| @@ -549,7 +551,7 @@ sub Timer_FW_Detail($$$$) { | |||
| $html.= "</tr>"; | |||
|
|
|||
| 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); | |||
There was a problem hiding this comment.
Ich denke, wir belassen die Hintergrundfarbe besser bei den vom Style vorgegeben Werten:
$html.= sprintf("<tr class="%s">", ($zeile & 1)?"odd":"even");
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Um diesen PR nicht ausversehen zu Mergen ist dieser Verzweis.
Hier wird #2 (comment) gerade vereinzelt separatisiert.
Angefangende Anpassungen #3.
|
Das hört sich sehr gut an
Von meinem iPhone gesendet
… Am 13.10.2019 um 19:41 schrieb HomeAutoUser ***@***.***>:
@HomeAutoUser commented on this pull request.
In TIMER_Sandro.pm:
> @@ -549,7 +551,7 @@ sub Timer_FW_Detail($$$$) {
$html.= "</tr>";
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.
|
|
mit #3 wurden die Anpassungen übernommen |
No description provided.