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

MultipleSelects mit per Default selektierten Optionen? #278

Closed
ischfr opened this issue Dec 14, 2022 · 11 comments
Closed

MultipleSelects mit per Default selektierten Optionen? #278

ischfr opened this issue Dec 14, 2022 · 11 comments

Comments

@ischfr
Copy link

ischfr commented Dec 14, 2022

Es scheint in der aktuellen MForm-Version 7.2.4 nicht möglich zu sein, bei einem MultipleSelect die Options gleich als "selected" zu definieren.

Anscheinend ist zwar – wie beim einfachen Select – ein "defaultValue"-String vorgesehen, ich konnte damit aber keine Auswirkungen auf ein Multiselect feststellen.

Ein einzelner Default-Wert würde hier auch nicht genügen, da ja potenziell mehrere Values ( bzw. deren Options) mit "selected" markiert werden sollen.

@alxndr-w
Copy link
Member

Hast du's mal kommagetrennt versucht, also "1,4,5" bzw. "Option1,Option2,Option3"?

@ischfr
Copy link
Author

ischfr commented Apr 30, 2024

Hab es gerade mal so versucht:

->addMultiSelectField(7, ['autoplay'=>'Automatisch abspielen', 'loop'=>'Endlosschleife', 'controls'=>'Player Bedienelemente', 'muted'=>'Ohne Sound'], ['label' => 'Abspiel Optionen', 'default-value' => 'loop, muted'])

Es ist die selbe Form, die ich auch bei einfachen Selects nutze. Habe mehrere Values wie vorgeschlagen kommagetrennt aufgeführt. Das hat aber leider keine Auswirkungen gehabt.

@alxndr-w
Copy link
Member

alxndr-w commented May 6, 2024

Und in Version 8? Das wäre jetzt vielleicht noch relevant zu fixen, bevor die neue Version online geht.

@joachimdoerr @skerbis

@ischfr
Copy link
Author

ischfr commented May 7, 2024

Mein Test von letzter Woche war übrigens bereits mit der Version 8 beta.

Ich bin halt immer noch nicht sicher, ob ich da grundsätzlich etwas falsch mache, oder ob default-value an dieser Stelle noch nie funktioniert hat.

Ich habe dazu für Multiselect m.W. auch noch nie Code-Beispiele gesehen.

@joachimdoerr
Copy link
Member

Default Value für Multiple Selects haben noch nie funktioniert. Ich hab das jetzt mal hinzugefügt, bitte Testen b8141ba

@ischfr
Copy link
Author

ischfr commented May 14, 2024

Ich hab es gerade mal mit meinem o.g. Code-Beispiel ausprobiert. Der erste Wert wird nun gesetzt, alles danach wird nach wie vor ignoriert. Ich schaffe es also nicht, mehrere Werte zu setzen. Oder muss ich die anders notieren?

@alxndr-w
Copy link
Member

Versuch es bitte ohne Leerzeichen nach dem Komma

@ischfr
Copy link
Author

ischfr commented May 14, 2024

ah! Das hat funktioniert! Vielen Dank, dann funktioniert es doch!
Das wäre vielleicht eine Erwähnung oder ein Beispiel in der Doc wert, dass man hier auf zusätzliche Leerzeichen achten muss.

@alxndr-w
Copy link
Member

Naja, eigentlich sind kommagetrennte Werte immer ohne Leerzeichen

@skerbis
Copy link
Member

skerbis commented May 14, 2024

Es ist durchaus möglich dass Werte mit Leerzeichen beginnen. Daher macht es genau das was es soll

@alxndr-w
Copy link
Member

So meinte ich das auch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants