fix insertText to do nothing in void nodes

lib/models/state.js

@@ -879,10 +879,15 @@ class State extends new Record(DEFAULTS) {
   insertText(text, marks) {
     let state = this
     let { cursorMarks, document, selection } = state
-    let after = selection
+    let after
+    const isVoid = document.hasVoidParent(state.startText)
 
     // Determine what the selection should be after inserting.
-    if (selection.isExpanded) {
+    if (isVoid) {
+      after = selection
+    }
+
+    else if (selection.isExpanded) {
       after = selection.collapseToStart().moveForward(text.length)
     }
 

lib/models/transforms.js

@@ -444,6 +444,11 @@ const Transforms = {
   insertTextAtRange(range, string, marks) {
     let node = this
 
+    // If inside a void node, do nothing.
+    const { startKey, startOffset } = range
+    const isVoid = node.hasVoidParent(startKey)
+    if (isVoid) return node
+
     // When still expanded, remove the current range first.
     if (range.isExpanded) {
       node = node.deleteAtRange(range)
@@ -451,7 +456,6 @@ const Transforms = {
     }
 
     // Insert text at the range's offset.
-    const { startKey, startOffset } = range
     let text = node.getDescendant(startKey)
     text = text.insertText(startOffset, string, marks)
     node = node.updateDescendant(text)

test/transforms/fixtures/insert-text-at-range/inside-void/index.js

@@ -0,0 +1,17 @@
+
+export default function (state) {
+  const { document, selection } = state
+  const texts = document.getTexts()
+  const first = texts.first()
+  const range = selection.merge({
+    anchorKey: first.key,
+    anchorOffset: 0,
+    focusKey: first.key,
+    focusOffset: 0
+  })
+
+  return state
+    .transform()
+    .insertTextAtRange(range, 'a')
+    .apply()
+}

test/transforms/fixtures/insert-text-at-range/inside-void/input.yaml

@@ -0,0 +1,5 @@
+
+nodes:
+  - kind: block
+    type: image
+    isVoid: true

test/transforms/fixtures/insert-text-at-range/inside-void/output.yaml

@@ -0,0 +1,5 @@
+
+nodes:
+  - kind: block
+    type: image
+    isVoid: true

test/transforms/fixtures/insert-text/inside-void/index.js

@@ -0,0 +1,27 @@
+
+import assert from 'assert'
+
+export default function (state) {
+  const { document, selection } = state
+  const texts = document.getTexts()
+  const first = texts.first()
+  const range = selection.merge({
+    anchorKey: first.key,
+    anchorOffset: 0,
+    focusKey: first.key,
+    focusOffset: 0
+  })
+
+  const next = state
+    .transform()
+    .moveTo(range)
+    .insertText('a')
+    .apply()
+
+  assert.deepEqual(
+    next.selection.toJS(),
+    range.toJS()
+  )
+
+  return next
+}

test/transforms/fixtures/insert-text/inside-void/input.yaml

@@ -0,0 +1,5 @@
+
+nodes:
+  - kind: block
+    type: image
+    isVoid: true

test/transforms/fixtures/insert-text/inside-void/output.yaml

@@ -0,0 +1,5 @@
+
+nodes:
+  - kind: block
+    type: image
+    isVoid: true