Werden gepostete patches eigentlich getestet bevor sie eingecheckt werden?
http://forum.fhem.de/index.php/topic,20564.msg167900.html#msg167900
Seit dem update gerade eben tut sich nichts mehr bei der Raumauswahl.
was genau funktioniert nicht?
ich mache die Auswahl gerade auf dem iPad möglich, da dort das Fenster nicht angezeigt wird.
Bin aber noch am Testen, die Ursache ist aber schon gefunden:
das iPad bekommt die falschen Positionswerte von jquery und der Dialog landet im nichts
ZitatWerden gepostete patches eigentlich getestet bevor sie eingecheckt werden?
Wohl auf Krawall gebuerstet...
Ja, ich teste es grundsaetzlich. Allerdings nicht mit allen Server-, Browser-, Style-, Modul- und Konfigurations-Kombinationen.
Bei mir tut es gerade ohne Probleme mit der demo config auf FF/Chrome/Safari/Android. iOS auch, hakt nur manchmal (refresh notwendig), und hat vermutlich nichts mit dem letzten Patch zu tun.
Solange es nicht gefixed ist, empfehle ich widgetOverride.
@Blackcat: das klingt komisch, weil ich nach einem refresh sowohl auf dem iPad wie auf dem iPhone das Dialog an richtiger Stelle gesehen habe.
Zitat von: rudolfkoenig am 13 Mai 2014, 12:44:47
@Blackcat: das klingt komisch, weil ich nach einem refresh sowohl auf dem iPad wie auf dem iPhone das Dialog an richtiger Stelle gesehen habe.
Da hat er dann wahrscheinlich die Bildschirmgröße bekommen...
mit dem CSS sieht man deutlich, dass es an der Position liegt, dann ist der dialog immer sofort da...
.ui-dialog {
top: 150px !important;
left: 300px !important;
}
Ich bin gerade dabei zu testen ob die Zuweisung der Position im Dialog Contructor etwas bringt. Ansonsten geht nur iPad im CSS als Sonderlocke betrachten
Zumal der Patch um die Gruppen- und nicht um die raumauswahl geht. Ich hab meinen Patch Vorschlag im Vorfeld mit Firefox und Chrome getestet. Jeweils Demo.cfg und echtsystem. Aber nur im darkstyle.
Ich hatte gestern den patch schon eingespielt und anschließend ging die Raum u. Gruppenauswahl nicht mehr. Nachdem ich im dev. Forum nicht posten kann, hab ich mich nicht gemeldet uns hab gedacht das wird dann schon passen.
Wollte nicht unhöflich klingen, nur nachfragen ob vorher nochmals getestet wird oder das beim Autor lieg.
hmm... an der Position scheint es wohl doch nicht zu liegen.
Ohne Änderung wieder weg... kommt mir so vor wie das "ios läd lange" Problem...
ggf. ist jquery noch nicht geladen?
ich schau mal
mist jquery wird wirklich nicht geladen
(siehe testfile mit Farben im Anhang)
bleibt auf orange stehen (also vor loadScript)
hm.. ich finde die Ursache nicht :'(
dafür habe ich einen Wunsch für die Formatierung mittels CSS:
table += '<tr>'+
'<td><div class="checkbox"><input name="'+v+'" type="checkbox"'+
(selObj[v] ? " checked" : "")+'/><label for="'+v+'"><span></span></label></div></td>'+
'<td><label for="'+v+'">'+v+'</label></td></tr>';
Zeile 34
@Blackcat: Bei "komischen" Vorschlaegen bitte auch eine Erklaerung mitliefern: wozu braucht man ein zweites label mit einem leeres span drin?
sagte ich eigentlich schon, dass ich javascript hasse? *g*
Zitat von: rudolfkoenig am 13 Mai 2014, 19:14:41
@Blackcat: Bei "komischen" Vorschlaegen bitte auch eine Erklaerung mitliefern: wozu braucht man ein zweites label mit einem leeres span drin?
iOS look a like switches, also für so was:
http://forum.fhem.de/index.php/topic,23406.msg168304.html#msg168304
das Label wird für die Hintergrundfarbe und der span für I oder O verwendet, das div definiert dann den IOS Schalter
@betateilchen: heute noch nicht.
@Blackcat:
1. loadScript bug in iOS: die alte longpoll Geschichte, faengt an mich zu nerven, da ich loadScript verstaerkt verwenden will. Habs "gefixed", hoffentlich tut es jetzt ohne Nebenwirkungen.
2. Zusaetzliche tags fuer multiple: Eigentlich gehoert sowas komplett ins CSS, aber bevor wir hier lange herumdiskutieren, habe ich es eingefuegt. Ich wollte dich zunaechst auf ::after/::before vertroesten, die koennen aber keine Tags, nur Text.
Danke Rudolf :)
*Daumen drück* das wird schon, wenn nicht muss man eben zum PC greifen ::)
Gibs dazu schon was neues?
Mittlerweile bin ich draufgekommen das nicht nur die Gruppen und Raum Auswahl bei mir nicht funktioniert sonder es wird gar kein Attribut in das Textfeld geladen. Wenn ich die die 01_FHEMWEB.pm r5809 einspiele klappt wieder alles. Getestet mit Safari und FireFox und default style.
Hab nur ich das Problem?
Grüße
Ich habe seid der Änderung von Rudolf keine Probleme mehr, auch nicht auf dem iPad
Ich hab gerade ein Update auf meinem FHEM2FHEM Raspi gemacht, da klappt auch wunderbar.
Anscheinend liegt es an meiner Konfiguration. Aber was kann das sein? :o
Hast du Räume mit Sonderzeichen Leerzeichen etc?
Ja, Leerzeichen und .
zB
1. Wohnzimmer
1.1 Plots
2. Schlafzimmer
2.1 Plots
Edit
Gerade am Raspi ausprobiert, da klappts auch mit einem Raum 1. Wohnzimmer
Nach ein bisschen Rumklicken bekomme ich diese Meldung im start terminal Fenster
^* matches null string many times in regex; marked by <-- HERE in m/^* <-- HERE .*$/ at fhem.pl line 3541.
^* matches null string many times in regex; marked by <-- HERE in m/^* <-- HERE .*$/ at fhem.pl line 3541.
Keine Ahnung ob das damit zusammenhängt, die Meldung kommt nicht regelmäßig.
Einer der event-on-changed-reading Attribute enthaelt etwas, was kein regexp ist.
Sollte mit dem erwaehnten Problem nicht zu tun haben.
Ich vermute due hast irgendein javascript Problem, was man z.Zt. nur in der JavaScript-Konsole sieht.
Ok danke dann werde ich mich mal auf die suche machen wo event-on-changed nicht passt.
JavaScript Konsole war der richtige Hinweis:
SyntaxError: missing ) after argument list
...emeines,An-#/#Abwesenheit,Arbeits#Dummy's,Arduino#Mega#Eingänge,Einstellungen,FH...
Anscheinend liegt es an dem ' im Gruppen Namen. Kann man das maskieren oder soll ich die Gruppe umbenennen?
Grüße
Habe ' und \ fuer alle Attribute in FHEMWEB geschuetzt, und kurz mit dem Raumauswahl getestet.
Bin auf Nebeneffekte gespannt.
Danke!