Skip to content

Change event not triggered for multi-select box on your site. #181

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
a2zbits opened this issue Nov 11, 2015 · 2 comments
Closed

Change event not triggered for multi-select box on your site. #181

a2zbits opened this issue Nov 11, 2015 · 2 comments

Comments

@a2zbits
Copy link

a2zbits commented Nov 11, 2015

I have noticed a minor issue with multi-select-box script on your site today. I don't know if this is some bug in VueJS or may be a browser issue. Please have a look at the attached error-screen first. Today, I was testing VueJS on following page of your site:
http://vuejs.org/guide/forms.html

Steps to Reproduce:

  • cilck the above link
  • select "A" in multi-select. this works and outputs "[A]".
  • Finally try to select all values by clicking "A" and drag the mouse far below "C". Release the mouse outside the boundary of multi-select. Now all values are selected, but VueJS could not realize the change event. VueJS will not realize this change even if you change the focus to some other field. It will realize this change only when you click the multi-select once again.

Not sure if this is VueJS bug or browser didn't fire change event. but whatever the case is, it will lead to unwanted results in back-end.

regards,

Ahmed Khan
a2zBits,
Lahore, Pakistan.
issue

@yyx990803
Copy link
Member

Thanks for reporting this. I've tested this in Chrome, Firefox, Safari and IE11 and it seems Safari is the only browser with this behavior. This is indeed because Safari is not firing the change event when the user releases the mouse outside of the select box. I think this is a Safari bug and you probably should report it at https://bugs.webkit.org/ .

@a2zbits
Copy link
Author

a2zbits commented Nov 12, 2015

Thank you for your prompt reply. I faced this issue in Google Chrome Version 46.0.2490.80 m. And amazingly I faced this issue only on your own site. when I copied the same script to a local test file this worked fine even in the same browser. so I think some CSS property of surrounding elements on your site may have caused this.

It is not a big issue. and yes we can close it.

dingyiming pushed a commit that referenced this issue Dec 21, 2016
dingyiming pushed a commit that referenced this issue Dec 21, 2016
Update list.md 大小写修改
kazupon pushed a commit to kazupon/vuejs.org that referenced this issue Oct 1, 2017
* Translate installation

* インストールの差分を翻訳 vuejs#126

* GithubをGitHubに統一 vuejs#126
lex111 pushed a commit to lex111/vuejs.org that referenced this issue Feb 3, 2018
isatrio pushed a commit to isatrio/vuejs.org that referenced this issue Nov 11, 2019
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

No branches or pull requests

2 participants