Hm. I'm worried that there could be other children of the contenteditable div in the DOM that wouldn't get removed if we only rekey the contents component.

I was wondering if there were any children not in the content="true" element @spicyj - thanks for the clarification.

Would a more targeted updating method work?

Right now I'm thinking:

• add a second key
• determine if we're inputting from a virtual-keyboard
• re-key only content="true" div if this is the case?
• else re-key everything

Is there any chance of this getting merged sometime soon?

Hey @codejet, the concern here is we're not sure if this is the proper fix. It fixes the keyboard issue, but because of the more targeted redraw we need to make sure it doesn't cause any other issues.

I don't think there should be any other children of the contentEditable div. As it is, I think I only pulled DraftEditorContents out into a separate component for shouldComponentUpdate purposes.

I'm actually wondering if it would always be safe to redraw only the DraftEditorContents when resetting the DOM.

• The reset behavior is only needed for cut and composition events.
• These events should only occur to contents that are definitely within the contentEditable.
• These events should only occur on descendants of DraftEditorContents.

The case where I can see problems occurring is where a cut or composition discards the entire DraftEditorContents for some reason.

Would you mind giving this a try and going through the cut/composition parts of the manual test plan to see if it works?

It seems like select all then cut would delete the contents completely…

Dear god, please merge this

FYI Googlers: I was able to sidestep this problem by by disabling autocorrect on the contentdeditable element on Android.
https://davidwalsh.name/disable-autocorrect

@hellendag Your suggestion about only redrawing DraftEditorContents works like a charm. Here's a PR for that approach: #907

May want to also check out #907, looks like it tackles the same issue with a similar fix.

We ended up merging a similar PR - #907 so closing this.