From 9f597c7ec0b30c8392a91967e3b4a19e72696368 Mon Sep 17 00:00:00 2001
From: Bruno Windels <brunow@matrix.org>
Date: Fri, 10 May 2019 14:25:13 +0200
Subject: [PATCH] no comment nodes without react,so can bring this back to
 simpler version

---
 src/editor/caret.js | 39 +++++++++++++--------------------------
 1 file changed, 13 insertions(+), 26 deletions(-)

diff --git a/src/editor/caret.js b/src/editor/caret.js
index a8fe3ddc68..f0359bed2d 100644
--- a/src/editor/caret.js
+++ b/src/editor/caret.js
@@ -68,35 +68,22 @@ function isVisibleNode(node) {
     return node.nodeType === Node.ELEMENT_NODE || node.nodeType === Node.TEXT_NODE;
 }
 
-function untilVisibleNode(node) {
-    // need to ignore comment nodes that react uses
-    while (node && !isVisibleNode(node)) {
-        node = node.nextSibling;
-    }
-    return node;
-}
-
 export function setCaretPosition(editor, caretPosition) {
-    let node = untilVisibleNode(editor.firstChild);
-    if (!node) {
-        node = editor;
-    } else {
-        let {index} = caretPosition;
-        while (node && index) {
-            node = untilVisibleNode(node.nextSibling);
-            --index;
-        }
-        if (!node) {
-            node = editor;
-        } else if (node.nodeType === Node.ELEMENT_NODE) {
-            // make sure we have a text node
-            node = node.childNodes[0];
-        }
-    }
     const sel = document.getSelection();
     sel.removeAllRanges();
     const range = document.createRange();
-    range.setStart(node, caretPosition.offset);
-    range.collapse(true);
+    let focusNode = editor.childNodes[caretPosition.index];
+    // node not found, set caret at end
+    if (!focusNode) {
+        range.selectNodeContents(editor);
+        range.collapse(false);
+    } else {
+        // make sure we have a text node
+        if (focusNode.nodeType === Node.ELEMENT_NODE) {
+            focusNode = focusNode.childNodes[0];
+        }
+        range.setStart(focusNode, caretPosition.offset);
+        range.collapse(true);
+    }
     sel.addRange(range);
 }