Skip to content

Multiple select elements @change event problem #3917

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

Closed
taskaz opened this issue Oct 12, 2016 · 3 comments · Fixed by #3922
Closed

Multiple select elements @change event problem #3917

taskaz opened this issue Oct 12, 2016 · 3 comments · Fixed by #3922
Assignees
Labels

Comments

@taskaz
Copy link

taskaz commented Oct 12, 2016

Multiple select box does not work in same Vue instance.

Vue.js version

  • Vue.js v2.0.2

Reproduction Link

https://jsfiddle.net/hd0drbo0/3/

Steps to reproduce

Change second select box value, method "testMethod" will be called infinite times, even it is not bonded to second select box.
Change first select box value, and later change second select box value, everything works as expected.

What is Expected?

Update of different models of select.

What is actually happening?

Infinite call of not assigned @change method.

@fnlctrl
Copy link
Member

fnlctrl commented Oct 12, 2016

Cannot reproduce on windows 10 x64 Chrome 53 / Firefox 48 / Edge, everything works fine here.
Can you specify the system and browser you're using?

@taskaz
Copy link
Author

taskaz commented Oct 12, 2016

Windows 10 x64, Chrome 53.0.

This happens only if at least one select element model is not initialized, existing value in select options is not specified in model by default.

Solution as suggested in line chat https://jsfiddle.net/crswll/hd0drbo0/5/

@fnlctrl
Copy link
Member

fnlctrl commented Oct 12, 2016

This happens only if at least one select element model is not initialized, existing value in select options is not specified in model by default.

Ok.. I see the problem now, looks like a bug to me.

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

Successfully merging a pull request may close this issue.

3 participants