Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update widget.js with rex:YForm_selectData #1484

Merged
merged 3 commits into from
Mar 7, 2024
Merged

Conversation

ynamite
Copy link
Contributor

@ynamite ynamite commented Mar 5, 2024

Fixes #1483

@skerbis
Copy link
Contributor

skerbis commented Mar 5, 2024

@dergel - getestet und behebt das Problem.

@ynamite
Copy link
Contributor Author

ynamite commented Mar 5, 2024

@skerbis bedingt Anpassungen bei mform (da neu native JS Implementierung erwünscht)
FriendsOfREDAXO/mform#307

@dergel dergel added this to the 4.2 milestone Mar 5, 2024
@dergel
Copy link
Member

dergel commented Mar 5, 2024

@ynamite Danke für die Anpassungen. :)

Wie das die AddOns implementieren, ist mir egal. Das kann jeder für sich entscheiden.

@ynamite
Copy link
Contributor Author

ynamite commented Mar 5, 2024

Damit wirst du aber Addon-Entwickler vor den Kopf stossen, da wie @skerbis schreibt, einige Addons davon abhängig sind und weiterhin auf jQuery setzen. Siehe:

Achtung es hat auch Auswirkung auf CKE5, mform, mblock, redactor, TinyMCE und Co. ... ich finde es nicht gut, dass wir jetzt sämtliche AddOns anpassen müssen. Bitte lasst das Jquery erstmal drin. Das bekommen wir so schnell nicht gefixt. @dergel Das muss vorbereitet sein. - Ich habe dafür zu wenig JS-Fähigkeiten und die Devs der betreffenden AddOns können das auch nicht adhoc anpassen. Gerade bei CKE5 müsste ein komplettes neues Build erzeugt werden.

#1483 (comment)

@skerbis
Copy link
Contributor

skerbis commented Mar 5, 2024

Puh! Danke. :-) Beim Release darauf hinweisen.
Vielleicht könnte man noch Tipps zur Umsetzung machen?
(Ich habe Null Peil davon)

@ynamite
Copy link
Contributor Author

ynamite commented Mar 5, 2024

Umsetzung ist fast identisch wie zuvor:

opener_id = 'unique-id';
window.addEventListener('rex:YForm_selectData_' + opener_id, (event) => {
        const id = event.detail.id;
        const value = event.detail.value;
        const multiple = event.detail.multiple;
        ...
})

zumindest im Falle von mform. Für andere Zwecke muss über lang oder kurz eine robustere Lösung her.
Sinnvoll wäre eh eine Art event bus für Redaxo, damit Events registriert werden können, die Redaxo dann auch wieder aufräumt. Momentan wird bei jedem mblock bzw. jeder mform ein neuer Event auf das Fenster gelegt, das kann schon zu Konflikten führen, wenn die nicht removed werden.

@dergel dergel merged commit 335dc5c into yakamara:master Mar 7, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

customLink ylink funktioniert nicht mehr mit yform 4.2.0
3 participants