Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enhancements for 0057-WebUI-ImprovedDutyCycleDisplay #675

Merged

Conversation

@ptweety
Copy link
Contributor

commented Jul 28, 2019

fixes #674

ptweety added 2 commits Jul 28, 2019
exchange visibility with display property
Signed-off-by: ptweety <me@ptweety.net>
fix colspan
Signed-off-by: ptweety <me@ptweety.net>
@ptweety

This comment has been minimized.

Copy link
Contributor Author

commented Jul 28, 2019

Hm, da ist was mit dem Zeichensatz im ersten Commit kaputt gegangen. Jetzt weiß ich leider noch nicht, wie ich das reparieren kann.

@jens-maus

This comment has been minimized.

Copy link
Owner

commented Jul 29, 2019

Danke für den PR. Kannst du bitte noch ein Screenshot der von dir angepassten DutyCycle Anzeige hier posten damit man die Änderung besser/schneller einsehen kann. Danke.

@ptweety

This comment has been minimized.

Copy link
Contributor Author

commented Jul 29, 2019

Gerne. Sichtbar ist das allerdings normalerweise gar nicht. Nur wenn man die Tabelle markiert oder mit extra css formatiert, dann sieht man die Spalten korrekt dargestellt (habe mal einen Fehler drin gelassen) und die überflüssigen Zeilen der unsichtbaren DutyCycles werden auch nicht gezeichnet (auch hier eines mal dringelassen)

Bildschirmfoto 2019-07-29 um 12 07 20

Bildschirmfoto 2019-07-29 um 12 12 20

@jens-maus jens-maus added this to the next release milestone Jul 30, 2019

@jens-maus jens-maus merged commit 89368fd into jens-maus:master Jul 30, 2019

@ptweety ptweety deleted the ptweety:0057-WebUI-ImprovedDutyCycleDisplay branch Jul 30, 2019

@jens-maus

This comment has been minimized.

Copy link
Owner

commented Aug 29, 2019

@ptweety BTW: Für den kommenden release musste ich deine Änderung teilweise wieder zurücknehmen (siehe 93b754e) bzw.:

https://homematic-forum.de/forum/viewtopic.php?f=65&t=52764#p525168

Anscheinend haben die colspan Anpassungen negative Effekte auf die Darstellung des Balkens. Wenn du das natürlich gerne noch anpassen willst kannst du gerne dann für den nächsten release noch einmal einen neuen PR schicken.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.