prosemirror-view 1.41.6 → 1.41.7

package/CHANGELOG.md

@@ -1,3 +1,11 @@
+## 1.41.7 (2026-03-18)
+
+### Bug fixes
+
+Work around a Chrome bug where it sometimes inserts a random `<br>` node when you press backspace before a widget.
+
+Fix a crash that could happen when deleting content during a node drag.
+
 ## 1.41.6 (2026-02-06)
 
 ### Bug fixes

package/dist/index.cjs

@@ -4413,7 +4413,13 @@ var DOMObserver = function () {
             var node = _step3.value;
             if (node.nodeName == "BR" && node.parentNode) {
               var after = node.nextSibling;
-              if (after && after.nodeType == 1 && after.contentEditable == "false") node.parentNode.removeChild(node);
+              while (after && after.nodeType == 1) {
+                if (after.contentEditable == "false") {
+                  node.parentNode.removeChild(node);
+                  break;
+                }
+                after = after.firstChild;
+              }
             }
           }
         } catch (err) {
@@ -5129,11 +5135,11 @@ var EditorView = function () {
     value: function updateDraggedNode(dragging, prev) {
       var sel = dragging.node,
         found = -1;
-      if (this.state.doc.nodeAt(sel.from) == sel.node) {
+      if (sel.from < this.state.doc.content.size && this.state.doc.nodeAt(sel.from) == sel.node) {
         found = sel.from;
       } else {
         var movedPos = sel.from + (this.state.doc.content.size - prev.doc.content.size);
-        var moved = movedPos > 0 && this.state.doc.nodeAt(movedPos);
+        var moved = movedPos > 0 && movedPos < this.state.doc.content.size && this.state.doc.nodeAt(movedPos);
         if (moved == sel.node) found = movedPos;
       }
       this.dragging = new Dragging(dragging.slice, dragging.move, found < 0 ? undefined : prosemirrorState.NodeSelection.create(this.state.doc, found));

package/dist/index.js

@@ -4730,8 +4730,13 @@ class DOMObserver {
             for (let node of added)
                 if (node.nodeName == "BR" && node.parentNode) {
                     let after = node.nextSibling;
-                    if (after && after.nodeType == 1 && after.contentEditable == "false")
-                        node.parentNode.removeChild(node);
+                    while (after && after.nodeType == 1) {
+                        if (after.contentEditable == "false") {
+                            node.parentNode.removeChild(node);
+                            break;
+                        }
+                        after = after.firstChild;
+                    }
                 }
         }
         else if (gecko && added.length) {
@@ -5578,12 +5583,12 @@ class EditorView {
     }
     updateDraggedNode(dragging, prev) {
         let sel = dragging.node, found = -1;
-        if (this.state.doc.nodeAt(sel.from) == sel.node) {
+        if (sel.from < this.state.doc.content.size && this.state.doc.nodeAt(sel.from) == sel.node) {
             found = sel.from;
         }
         else {
             let movedPos = sel.from + (this.state.doc.content.size - prev.doc.content.size);
-            let moved = movedPos > 0 && this.state.doc.nodeAt(movedPos);
+            let moved = movedPos > 0 && movedPos < this.state.doc.content.size && this.state.doc.nodeAt(movedPos);
             if (moved == sel.node)
                 found = movedPos;
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prosemirror-view",
-  "version": "1.41.6",
+  "version": "1.41.7",
   "description": "ProseMirror's view component",
   "type": "module",
   "main": "dist/index.cjs",

package/src/domobserver.ts

@@ -197,8 +197,13 @@ export class DOMObserver {
       // backspace out the last bit of text before an inline-flex node (#1552)
       for (let node of added) if (node.nodeName == "BR" && node.parentNode) {
         let after = node.nextSibling
-        if (after && after.nodeType == 1 && (after as HTMLElement).contentEditable == "false")
-          node.parentNode.removeChild(node)
+        while (after && after.nodeType == 1) {
+          if ((after as HTMLElement).contentEditable == "false") {
+            node.parentNode.removeChild(node)
+            break
+          }
+          after = after.firstChild
+        }
       }
     } else if (browser.gecko && added.length) {
       let brs = added.filter(n => n.nodeName == "BR") as HTMLElement[]

package/src/index.ts

@@ -274,11 +274,11 @@ export class EditorView {
 
   private updateDraggedNode(dragging: Dragging, prev: EditorState) {
     let sel = dragging.node!, found = -1
-    if (this.state.doc.nodeAt(sel.from) == sel.node) {
+    if (sel.from < this.state.doc.content.size && this.state.doc.nodeAt(sel.from) == sel.node) {
       found = sel.from
     } else {
       let movedPos = sel.from + (this.state.doc.content.size - prev.doc.content.size)
-      let moved = movedPos > 0 && this.state.doc.nodeAt(movedPos)
+      let moved = movedPos > 0 && movedPos < this.state.doc.content.size && this.state.doc.nodeAt(movedPos)
       if (moved == sel.node) found = movedPos
     }
     this.dragging = new Dragging(dragging.slice, dragging.move,