search

yakamara / yform

YForm für REDAXO 5 – Formulare im Frontend und Backend mit Verwaltung von Datenbank-Tabellen.
MIT License
77 stars 55 forks source link

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

Open tyrant88 opened 4 months ago

tyrant88 commented 4 months ago

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

Zwischenablage-1

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

ynamite commented 2 weeks ago

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 commented 2 weeks ago

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.

grafik
ynamite commented 2 weeks ago

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 commented 2 weeks ago

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 commented 2 weeks ago

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

tyrant88 commented 2 weeks ago

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

ynamite commented 2 weeks ago

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: https://github.com/yakamara/yform/blob/ada7342777994d8f6983c2a8eb127e7de54ac460/assets/widget.js#L172

tyrant88 commented 2 weeks ago

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