Raumauswahl funktioniert nicht.

[fhainz]

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.

[Blackcat]

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

[rudolfkoenig]

Werden 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.

[Blackcat]

@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...

Code Auswählen Erweitern


.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

[svenson08]

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.

[fhainz]

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.

[Blackcat]

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

[Blackcat]

mist jquery wird wirklich nicht geladen
(siehe testfile mit Farben im Anhang)
bleibt auf orange stehen (also vor loadScript)

[Blackcat]

hm.. ich finde die Ursache nicht 

dafür habe ich einen Wunsch für die Formatierung mittels CSS:

Code Auswählen Erweitern

        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

[rudolfkoenig]

@Blackcat: Bei "komischen" Vorschlaegen bitte auch eine Erklaerung mitliefern: wozu braucht man ein zweites label mit einem leeres span drin?

[betateilchen]

sagte ich eigentlich schon, dass ich javascript hasse? *g*

[Blackcat]

@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

[rudolfkoenig]

@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.

[Blackcat]

Danke Rudolf

*Daumen drück* das wird schon, wenn nicht muss man eben zum PC greifen 

[fhainz]

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