Emiting change on blur about react-contenteditable HOT 5 CLOSED

Hi,

@mertkahyaoglu theinitial code comes from this SO question:
http://stackoverflow.com/questions/22677931/react-js-onchange-event-for-contenteditable

I don't know why @brigand did put a onBlur listener, but in my experience I don't see much value in keeping it.

My current UncontrolledContentEditable I use in production looks like this and does not been to emit change on blur:
https://gist.github.com/slorber/b84c6c7c2d2c3c6250c53a001b674688

from react-contenteditable.

@slorber Thanks for quick response. Actually #16 fixes the problem I guess, but it is not available on npm (still 1.1.0). With that PR, I can override the onBlur event by passing an empty function to prevent emiting change.

from react-contenteditable.

Hello everyone. Here are my warm regards to everyone using contentEditable in general and contentEditable with React in particular.

Here's a question: considering that the current solution won't work on IE11 because of this and by definition (I confess that I didn't take a look at the React SynteticEvent creation based on DOM events), what do you guys think about using onKeyUp instead of onInput here?.

Kind regards and thank you for your work.

from react-contenteditable.

Hmmm I've just done some test and for me it seems onInput works fine on IE, however I've been reported there might be a problem on my app on IE11 where a text paste does not trigger onInput.

I'll have to digg deeper another day

from react-contenteditable.

The original onBlur issue seems resolved, so closing this one. The question about onInput events in IE11 may be worth opening as a separate issue, if it is still relevant.

from react-contenteditable.