be_manager_relation popup select multiple schließt sofort nach der ersten Auswahl das Fenster.

[tyrant88]

Wie im Titel beschrieben:
Popup im Backend in einem Feld. Man wählt einen Datensatz aus mit "übernehmen" und das Fenster schließt sich sofort.

Ich blicke das js Event-System nicht so ganz.
Daher habe ich laienhaft eine Prüfung auf "multiple" in diese Zeile hinzugefügt
https://github.com/yakamara/yform/blob/master/plugins/manager/assets/widget.js#L158

Funzt.
Aber wie ist das eigentlich gedacht? Soll das preventDefault beim Dispatchen des Events getriggert werden?

[ynamite]

Ich rate mal: wenn die Methode, die durch den Event im opener getriggert wird, nicht abschliesst, soll das Popup geöffnet bleiben. Braucht es vermutlich nicht unbedingt.

[christophboecker]

Irgendwas ist seit der letzten Änderung mau. Verstehen tue ich es nicht. Der grüne Teil ist neu, sonst nix. Zumindest nicht in dieser Datei.

[ynamite]

War das nicht schon immer so? @christophboecker
Also das sich das Fenster nach der ersten Auswahl schliesst?
Die Änderung wurde gemacht, um sich von jQuery zu lösen.

[christophboecker]

Bis 4.1.2 konnte man zwischen Einzelauswahl und Mehrfachauswahl unterscheiden. Das galt auch noch für die Betas (ich habe für das Bild oben 4.1.2-beta1 genommen. Seit 4.2 ist das Mehrfach-Feature futsch.

[ynamite]

Wenn der fix von @tyrant88 funzt, dann gerne PR.

[tyrant88]

Funzen tut es, aber ich wollte ja noch jemand mit js Expertise dazu hören...

[ynamite]

Der überarbeitete code in widget.js stammt von mir.
Ist ein Weilchen her und ich weiss gerade nicht mehr, weshalb das defaultPrevented nötig war. Dein fix sieht aber gut aus, wenn du es getestet hast und es funzt, dann dürfte das passen ;)

Sehe gerade, dass das bereits in der master branch drinne ist, siehe:

yform/assets/widget.js

Line 172 in ada7342

if (multiple !== "1") {

[tyrant88]

Ah, stimmt. Dann nur noch warten auf das nächste Release.