FHEM Forum

FHEM => Frontends => Thema gestartet von: Blackcat am 04 Juni 2014, 12:57:29

Titel: Bugfix für verschobene Headerleiste bei Befehlen
Beitrag von: Blackcat am 04 Juni 2014, 12:57:29
Hallo ihr Lieben,

nachdem mehreren Usern aufgefallen ist, dass bei Styles, die keine absolute Position des Headers setzen, der Header nach unten rutscht. Habe ich nach der Ursache gesucht:

Das HTML Dokument erzeugt für cmds erst den content und dann den Rest. Lösung wäre deshalb den Aufruf der Headerinit einfach vor die cmd Logik zu schieben:

Index: C:/Users/ohmayer/Desktop/Daten/FHEM/FHEM/01_FHEMWEB.pm
===================================================================
--- C:/Users/ohmayer/Desktop/Daten/FHEM/FHEM/01_FHEMWEB.pm (revision 6053)
+++ C:/Users/ohmayer/Desktop/Daten/FHEM/FHEM/01_FHEMWEB.pm (working copy)
@@ -661,7 +661,9 @@
     $FW_cmdret = $FW_activateInform = "";
     $cmd = "style eventMonitor";
   }
-

+  FW_roomOverview($cmd);

   if($FW_cmdret) {
     $FW_detail = "";
     $FW_room = "";
@@ -693,7 +695,6 @@

   }

-  FW_roomOverview($cmd);
   if($FW_contentFunc) {
     no strict "refs";
     my $ret = &{$FW_contentFunc}($arg);


Kann das jemand noch prüfen und ggf. einchecken, oder mir die Freigabe zum einchecken geben? Ist ja nicht meine Ecke :)

PS: Beispiel des Fehlverhaltens:
http://forum.fhem.de/index.php/topic,23406.msg174004.html#msg174004
Titel: Antw:Bugfix für verschobene Headerleiste bei Befehlen
Beitrag von: maxritti am 04 Juni 2014, 16:50:38
Hi Sandra,

gerade habe ich mal Deinen Patch in meiner Entwicklungsumgebung eingespielt.
Das sieht gut aus.
Nun verschwindet nichts mehr an den unteren Rand.

Wobei ich diese 01_FHEMWEB.pm habe.

# $Id: 01_FHEMWEB.pm 6031 2014-06-02 07:19:40Z rudolfkoenig $

Oder hat die Revision 6053 in deinem Patchfile nichts mit der $Id zu tun?

Beim einchecken kann ich allerdings nicht helfen  ;)
Titel: Antw:Bugfix für verschobene Headerleiste bei Befehlen
Beitrag von: Blackcat am 04 Juni 2014, 17:00:44
Zitat von: maxritti am 04 Juni 2014, 16:50:38
Wobei ich diese 01_FHEMWEB.pm habe.

# $Id: 01_FHEMWEB.pm 6031 2014-06-02 07:19:40Z rudolfkoenig $

Oder hat die Revision 6053 in deinem Patchfile nichts mit der $Id zu tun?

Hm.. habe vorher extra geupdatet ... denke mal die Id ist vielleicht eine andere?
Aber Rudolf wird wahrscheinlich die Änderungen einspielen, da er alle commits gemacht hatte, daher denke ich wird er das mit der Id wissen (kann ja auch sein, dass mein SVN das diff file nicht richtig benannt hat) ;)
Titel: Antw:Bugfix für verschobene Headerleiste bei Befehlen
Beitrag von: betateilchen am 04 Juni 2014, 17:31:47
Die 6053 (aktueller SVN Stand zum Zeitpunkt der Erstellung des patch) aus dem patch hat nichts direkt mit der 6031 der tatsächlichen FHEMWEB Version zu tun. Kein Grund zur Sorge.
Titel: Antw:Bugfix für verschobene Headerleiste bei Befehlen
Beitrag von: rudolfkoenig am 04 Juni 2014, 22:12:26
Habs eingecheckt.
Titel: Antw:Bugfix für verschobene Headerleiste bei Befehlen
Beitrag von: Blackcat am 04 Juni 2014, 22:29:26
Danke :)