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

Fix onChange behavior on ie11 #130

Closed
wants to merge 1 commit into from
Closed

Fix onChange behavior on ie11 #130

wants to merge 1 commit into from

Conversation

adrianton3
Copy link

onChange doesn't trigger correctly on IE11 in react 15 for inputs/textareas. onInput seems to work fine - I tested in IE10, 11, Chrome, Firefox and Safari.
There is a fix in the working here facebook/react#8575 - also a good source of info on related issues

@Andarist
Copy link
Owner

Do u know when this regression in React has happened? Wouldnt like to include this as its quite a big internal change. Are there any explored alternatives? Also - what behaviour exactly is it fixing? Could u provide a shareable example of pre and post PR behaviours?

@Andarist
Copy link
Owner

Please also check against the newest version of the library, Im testing it right now with controlled input and everything (regarding onChange) seems fine on IE11 (and other browsers)

@Andarist
Copy link
Owner

@adrianton3 ive tested the newest version on IE 11 and everything seems fine, if you still experience the issue - could u post a reproduced example so I can test and debug it? going to close this for now but feel free to post back to me

@Andarist Andarist closed this May 20, 2017
@peterkeating
Copy link

Was experiencing this issue, was resolved by updating to the latest version (4.3.2).

@Andarist
Copy link
Owner

Andarist commented Jun 6, 2017

@peterkeating thanks for reporting back!

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

Successfully merging this pull request may close these issues.

3 participants