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.

Sign up for GitHub

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

Jump to bottom

<select :multiple="multiple">: if multiple value is undefined, the attribute is enabled #4855

Closed
sellvana opened this issue Feb 5, 2017 · 2 comments
Closed

<select :multiple="multiple">: if multiple value is undefined, the attribute is enabled #4855

sellvana opened this issue Feb 5, 2017 · 2 comments
Assignees
@posva
Labels
improvement

Comments

@sellvana
Copy link

sellvana commented Feb 5, 2017

Hello, I think that undefined values should be handled as falsy values in attributes.

https://jsfiddle.net/0yzf8wgf/

Thanks.
@posva posva self-assigned this Feb 5, 2017
@posva
Copy link
Member

posva commented Feb 5, 2017

Thanks for pointing it out!

posva added a commit to posva/vue that referenced this issue Feb 5, 2017
@posva
Fix multiple attr in select with undefined value
30bef57 
Fix vuejs#4855
@posva posva mentioned this issue Feb 5, 2017
Merged
@sellvana
Copy link
Author

sellvana commented Feb 5, 2017

Thank you for fixing!

yyx990803 pushed a commit that referenced this issue Feb 6, 2017
@posva @yyx990803
Fix multiple attr in select with undefined value (#4859)
ec7fca8 
Fix #4855
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@posva posva

Labels
improvement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@posva @sellvana