-
Notifications
You must be signed in to change notification settings - Fork 78
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
:list field, fake placeholder and empty value #109
Comments
It might be an idea to add a separate attribute that would be preferred to the |
I also had a related problem where I would end up with duplicate keys because the option values were the same. Here's the version I wrote which supports the
|
I'd be open to a pr to allow providing an explicit |
Hello,
I would like to have the possibility to add a fake placeholder to select lists.
I found this : http://stackoverflow.com/questions/5805059/how-do-i-make-a-placeholder-for-a-select-box
So with reagent 0.5.1 and reagent-forms 0.5.22, this was working :
But React/Reagent was throwing a warning:
Warning: Every element in a seq should have a unique :key : ([:option {:key nil, :disabled true, :hidden true}...
After upgrading to reagent 0.6.0-rc, this doesn't work anymore. The first value of the sequence is displayed by default, no fake placeholder, and the value of the ratom (:countries here) is not selected when provided.
I tried this and removed the warning but it didn't work better as a bound form (with
[bind-fields ... ...]
):Is having the "key" attribute used for the React key and for the value a good thing? Here I really want and empty value so that
null
is returned to the API.Thanks
The text was updated successfully, but these errors were encountered: