Android related improvements (#5315)

* Allow consumer handling of `onInput` event

* Avoid restoring the DOM for characterData mutations

.changeset/android-text-mutations.md

@@ -0,0 +1,5 @@
+---
+'slate-react': patch
+---
+
+The `RestoreDOM` manager that is used Android no longer restores the DOM to its previous state for text mutations. This allows the editor state to be reconciled during a composition without interrupting the composition, as programatically updating the `textContent` of a text node ends the current composition.

.changeset/consumer-oninput-event.md

@@ -0,0 +1,5 @@
+---
+'slate-react': patch
+---
+
+Fixed consumer defined `onInput` event handler not being invoked when passed to the `<Editable>` component.

packages/slate-react/src/components/editable.tsx

@@ -894,7 +894,11 @@ export const Editable = (props: EditableProps) => {
               },
               [readOnly]
             )}
-            onInput={useCallback((event: React.SyntheticEvent) => {
+            onInput={useCallback((event: React.FormEvent<HTMLDivElement>) => {
+              if (isEventHandled(event, attributes.onInput)) {
+                return
+              }
+
               if (androidInputManager) {
                 androidInputManager.handleInput()
                 return

packages/slate-react/src/components/restore-dom/restore-dom-manager.ts

@@ -31,23 +31,26 @@ export const createRestoreDomManager = (
   }
 
   function restoreDOM() {
-    bufferedMutations.reverse().forEach(mutation => {
+    if (bufferedMutations.length > 0) {
-      if (mutation.type === 'characterData') {
+      bufferedMutations.reverse().forEach(mutation => {
-        mutation.target.textContent = mutation.oldValue
+        if (mutation.type === 'characterData') {
-        return
+          // We don't want to restore the DOM for characterData mutations
-      }
+          // because this interrupts the composition.
+          return
+        }
 
-      mutation.removedNodes.forEach(node => {
+        mutation.removedNodes.forEach(node => {
-        mutation.target.insertBefore(node, mutation.nextSibling)
+          mutation.target.insertBefore(node, mutation.nextSibling)
+        })
+
+        mutation.addedNodes.forEach(node => {
+          mutation.target.removeChild(node)
+        })
       })
 
-      mutation.addedNodes.forEach(node => {
+      // Clear buffered mutations to ensure we don't undo them twice
-        mutation.target.removeChild(node)
+      clear()
-      })
+    }
-    })
-
-    // Clear buffered mutations to ensure we don't undo them twice
-    clear()
   }
 
   return {