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

Fehlerhafte Spaltenbreite #10

Closed
bismosa opened this issue Aug 7, 2019 · 9 comments
Closed

Fehlerhafte Spaltenbreite #10

bismosa opened this issue Aug 7, 2019 · 9 comments

Comments

@bismosa
Copy link
Collaborator

bismosa commented Aug 7, 2019

Hallo!

Um es übersichtlicher zu halten mache ich mal eine neue Issue auf...

Es geht um die Fehlerhafte Spaltenbreite (Siehe ToDo-List).
Bisher haben wir, um die UI anzuzeigen, die "FW_summaryFn" benutzt um statt des Status-Icon die UI mit anzuzeigen.
Dies macht bei bestimmten Designs Schwierigkeiten.
Um dies zu umgehen könnte man die Detail-Ansicht mit dem HTML-Code erzeugen und diese dann extra einblenden lassen:

Detailansicht erzeugen:
$hash->{FW_detailFn} = "SD_Keeloq::summaryFn";
Detailansicht in der Raumansicht hinzufügen
$hash->{FW_addDetailToSummary} = 1;
Device-Overview einblenden:
$hash->{FW_deviceOverview} = 1;
Es muss die folgende Zeile entfernt werden:
$hash->{FW_summaryFn} = "SD_Keeloq::summaryFn"; # displays html instead of status icon in fhemweb room-view

Siehe auch: https://forum.fhem.de/index.php?topic=71772.0

Dadurch ändert sich das Design geringfügig aber sollte dann auch bei allen FHEM-Designs funktionieren:
Alt:
image
Neu:
image

ggf. könnte man nun noch den HTML-Code anpassen, um schmalere Spalten zu bekommen. (Abhängig derzeit von anderen Devices im gleichen Raum)

Gruß
Bismosa

@HomeAutoUser
Copy link
Collaborator

Ich fasse mal deinen Post ebenso kuzr auf.
Sehe ich das richtig, das die Zeile

$hash->{FW_summaryFn} = "SD_Keeloq::summaryFn"; # displays html instead of status icon in fhemweb room-view

entfernt oder auskommentiert werden muss um die richtige Spaltendarstellung (gleichmäßig) zu erziehlen?

MfG

@bismosa
Copy link
Collaborator Author

bismosa commented Aug 8, 2019

Hallo!

ja, diese muss entfernt oder auskommentiert werden. Sonst wird der HTML-Kram (also die Buttons) doppelt angezeigt.

Gruß
Bismosa

HomeAutoUser added a commit to HomeAutoUser/RFFHEM that referenced this issue Aug 8, 2019
@HomeAutoUser
Copy link
Collaborator

Ich habe es bei uns eingecheckt.

@bismosa
Copy link
Collaborator Author

bismosa commented Aug 8, 2019

Hallo,

wenn ich da gerade richtig gesehen habe, hast Du nun nur die eine Zeile auskommentiert? Diese Zeilen müssen aber noch hinzugefügt werden:

$hash->{FW_detailFn} = "SD_Keeloq::summaryFn";
$hash->{FW_addDetailToSummary} = 1;
$hash->{FW_deviceOverview} = 1;

Gruß
Bismosa

@HomeAutoUser
Copy link
Collaborator

Das ging nicht hervor :D

@bismosa
Copy link
Collaborator Author

bismosa commented Aug 8, 2019

Sorry :)

Gruß
Bismosa

@HomeAutoUser
Copy link
Collaborator

Bitte nochmal kontrollieren
Änderung 1
RFD-FHEM/RFFHEM@2a63d61

Änderung 2
RFD-FHEM/RFFHEM@e9e62a8

@bismosa
Copy link
Collaborator Author

bismosa commented Aug 8, 2019

Jup. So sollte es passen :)
DANKE! :)

Gruß
Bismosa

HomeAutoUser added a commit to RFD-FHEM/RFFHEM that referenced this issue Aug 9, 2019
* 14_SD_Keeloq.pm - fixed
- fixed wrong Displayview fhem/SD_Keeloq#10
- fixed autocreate serial´s without 00 fhem/SD_Keeloq#6 https://forum.fhem.de/index.php/topic,13596.msg962950.html#msg962950
@HomeAutoUser
Copy link
Collaborator

Da nun der Fix eingearbeitet ist und in der Version gefixt, so würde ich den Faden schließen.
Sollte denn noch was sein, einfach ReOpen oder hier weiter schreiben.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants