@codemirror/view 6.39.2 → 6.39.4

package/CHANGELOG.md

@@ -1,3 +1,21 @@
+## 6.39.4 (2025-12-12)
+
+### Bug fixes
+
+Fix a bug where paste events handlers on Chrome could fail to run when pasting on a blank line.
+
+Fix a regression causing the native cursor to get stuck before block widgets with side>0.
+
+Fix a crash in content DOM building after a block widget.
+
+Fix a bug in `posAtCoords` that would in some circumstances make it return positions on the wrong side of a block widget.
+
+## 6.39.3 (2025-12-11)
+
+### Bug fixes
+
+Fix a bug that could corrupt the rendered document in some situations involving adjacent mark decorations of the same type.
+
 ## 6.39.2 (2025-12-09)
 
 ### Bug fixes

package/dist/index.cjs

@@ -1991,7 +1991,7 @@ class LineTile extends CompositeTile {
             if (this.dom.contains(tile.dom)) {
                 if (tile.isText())
                     return new DOMPos(tile.dom, Math.min(tile.dom.nodeValue.length, offset));
-                return tile.domPosFor(offset, side);
+                return tile.domPosFor(offset, tile.flags & 16 /* TileFlag.Before */ ? 1 : tile.flags & 32 /* TileFlag.After */ ? -1 : side);
             }
             let parent = found.tile.parent, saw = false;
             for (let ch of parent.children) {
@@ -2623,12 +2623,14 @@ class TileUpdate {
                             this.builder.addBlockWidget(widget);
                         }
                         else {
+                            this.builder.ensureLine(null);
                             this.builder.addInlineWidget(widget, activeMarks, openMarks);
                             openMarks = activeMarks.length;
                         }
                     }
                 }
                 else if (tile.isText()) {
+                    this.builder.ensureLine(null);
                     if (!from && to == tile.length) {
                         this.builder.addText(tile.text, activeMarks, openMarks, this.cache.reuse(tile));
                     }
@@ -2647,8 +2649,10 @@ class TileUpdate {
                     this.cache.add(tile);
                 }
                 else if (tile instanceof MarkTile) {
+                    this.builder.ensureLine(null);
                     this.builder.addMark(tile, activeMarks, openMarks);
                     this.cache.reused.set(tile, 1 /* Reused.Full */);
+                    openMarks = activeMarks.length;
                 }
                 else {
                     return false;
@@ -2661,10 +2665,8 @@ class TileUpdate {
                 }
                 else {
                     this.cache.add(tile);
-                    if (tile instanceof MarkTile) {
+                    if (tile instanceof MarkTile)
                         activeMarks.unshift(tile.mark);
-                        openMarks++;
-                    }
                 }
                 this.openWidget = false;
             },
@@ -3075,7 +3077,7 @@ class DocView {
         let { anchorNode, anchorOffset } = view.observer.selectionRange;
         if (!sel || !cursor.empty || !cursor.assoc || !sel.modify)
             return;
-        let line = this.lineAt(cursor.head);
+        let line = this.lineAt(cursor.head, cursor.assoc);
         if (!line)
             return;
         let lineStart = line.posAtStart;
@@ -3154,7 +3156,7 @@ class DocView {
         let after, afterOff = -1, afterBad = false;
         this.tile.blockTiles((tile, off) => {
             if (tile.isWidget()) {
-                if ((tile.flags & 32 /* TileFlag.After */) && before)
+                if ((tile.flags & 32 /* TileFlag.After */) && off >= pos)
                     return true;
                 if (tile.flags & 16 /* TileFlag.Before */)
                     beforeBad = true;
@@ -3192,8 +3194,8 @@ class DocView {
         }
         return tile.coordsIn(offset, side);
     }
-    lineAt(pos) {
-        let { tile } = this.tile.resolveBlock(pos, 1);
+    lineAt(pos, side) {
+        let { tile } = this.tile.resolveBlock(pos, side);
         return tile.isLine() ? tile : null;
     }
     coordsForChar(pos) {
@@ -3739,7 +3741,10 @@ function posAtCoords(view, coords, precise, scanY) {
     if (block.type != exports.BlockType.Text)
         return yOffset < (block.top + block.bottom) / 2 ? new PosAssoc(block.from, 1) : new PosAssoc(block.to, -1);
     // Here we know we're in a line, so run the logic for inline layout
-    return posAtCoordsInline(view, view.docView.lineAt(block.from), block.from, x, y);
+    let line = view.docView.lineAt(block.from, 2);
+    if (!line || line.length != block.length)
+        line = view.docView.lineAt(block.from, -2);
+    return posAtCoordsInline(view, line, block.from, x, y);
 }
 // Scan through the rectangles for the content of a tile, finding the
 // one closest to the given coordinates, prefering closeness in Y over
@@ -4658,7 +4663,8 @@ function eventBelongsToEditor(view, event) {
     if (event.defaultPrevented)
         return false;
     for (let node = event.target, tile; node != view.contentDOM; node = node.parentNode)
-        if (!node || node.nodeType == 11 || ((tile = Tile.get(node)) && tile.isWidget() && tile.widget.ignoreEvent(event)))
+        if (!node || node.nodeType == 11 ||
+            ((tile = Tile.get(node)) && tile.isWidget() && !tile.isHidden && tile.widget.ignoreEvent(event)))
             return false;
     return true;
 }
@@ -4777,42 +4783,13 @@ function rangeForClick(view, pos, bias, type) {
         return groupAt(view.state, pos, bias);
     }
     else { // Triple click
-        let visual = view.docView.lineAt(pos), line = view.state.doc.lineAt(visual ? visual.posAtEnd : pos);
+        let visual = view.docView.lineAt(pos, bias), line = view.state.doc.lineAt(visual ? visual.posAtEnd : pos);
         let from = visual ? visual.posAtStart : line.from, to = visual ? visual.posAtEnd : line.to;
         if (to < view.state.doc.length && to == line.to)
             to++;
         return state.EditorSelection.range(from, to);
     }
 }
-let inside = (x, y, rect) => y >= rect.top && y <= rect.bottom && x >= rect.left && x <= rect.right;
-// Try to determine, for the given coordinates, associated with the
-// given position, whether they are related to the element before or
-// the element after the position.
-function findPositionSide(view, pos, x, y) {
-    let line = view.docView.lineAt(pos);
-    if (!line)
-        return 1;
-    let off = pos - line.posAtStart;
-    // Line boundaries point into the line
-    if (off == 0)
-        return 1;
-    if (off == line.length)
-        return -1;
-    // Positions on top of an element point at that element
-    let before = line.coordsIn(off, -1);
-    if (before && inside(x, y, before))
-        return -1;
-    let after = line.coordsIn(off, 1);
-    if (after && inside(x, y, after))
-        return 1;
-    // This is probably a line wrap point. Pick before if the point is
-    // above its bottom.
-    return before && before.bottom >= y ? -1 : 1;
-}
-function queryPos(view, event) {
-    let pos = view.posAtCoords({ x: event.clientX, y: event.clientY }, false);
-    return { pos, bias: findPositionSide(view, pos, event.clientX, event.clientY) };
-}
 const BadMouseDetail = browser.ie && browser.ie_version <= 11;
 let lastMouseDown = null, lastMouseDownCount = 0, lastMouseDownTime = 0;
 function getClickType(event) {
@@ -4825,7 +4802,7 @@ function getClickType(event) {
         Math.abs(last.clientY - event.clientY) < 2) ? (lastMouseDownCount + 1) % 3 : 1;
 }
 function basicMouseSelection(view, event) {
-    let start = queryPos(view, event), type = getClickType(event);
+    let start = view.posAndSideAtCoords({ x: event.clientX, y: event.clientY }, false), type = getClickType(event);
     let startSel = view.state.selection;
     return {
         update(update) {
@@ -4835,10 +4812,10 @@ function basicMouseSelection(view, event) {
             }
         },
         get(event, extend, multiple) {
-            let cur = queryPos(view, event), removed;
-            let range = rangeForClick(view, cur.pos, cur.bias, type);
+            let cur = view.posAndSideAtCoords({ x: event.clientX, y: event.clientY }, false), removed;
+            let range = rangeForClick(view, cur.pos, cur.assoc, type);
             if (start.pos != cur.pos && !extend) {
-                let startRange = rangeForClick(view, start.pos, start.bias, type);
+                let startRange = rangeForClick(view, start.pos, start.assoc, type);
                 let from = Math.min(startRange.from, range.from), to = Math.max(startRange.to, range.to);
                 range = from < range.from ? state.EditorSelection.range(from, to) : state.EditorSelection.range(to, from);
             }

package/dist/index.js

@@ -1987,7 +1987,7 @@ class LineTile extends CompositeTile {
             if (this.dom.contains(tile.dom)) {
                 if (tile.isText())
                     return new DOMPos(tile.dom, Math.min(tile.dom.nodeValue.length, offset));
-                return tile.domPosFor(offset, side);
+                return tile.domPosFor(offset, tile.flags & 16 /* TileFlag.Before */ ? 1 : tile.flags & 32 /* TileFlag.After */ ? -1 : side);
             }
             let parent = found.tile.parent, saw = false;
             for (let ch of parent.children) {
@@ -2619,12 +2619,14 @@ class TileUpdate {
                             this.builder.addBlockWidget(widget);
                         }
                         else {
+                            this.builder.ensureLine(null);
                             this.builder.addInlineWidget(widget, activeMarks, openMarks);
                             openMarks = activeMarks.length;
                         }
                     }
                 }
                 else if (tile.isText()) {
+                    this.builder.ensureLine(null);
                     if (!from && to == tile.length) {
                         this.builder.addText(tile.text, activeMarks, openMarks, this.cache.reuse(tile));
                     }
@@ -2643,8 +2645,10 @@ class TileUpdate {
                     this.cache.add(tile);
                 }
                 else if (tile instanceof MarkTile) {
+                    this.builder.ensureLine(null);
                     this.builder.addMark(tile, activeMarks, openMarks);
                     this.cache.reused.set(tile, 1 /* Reused.Full */);
+                    openMarks = activeMarks.length;
                 }
                 else {
                     return false;
@@ -2657,10 +2661,8 @@ class TileUpdate {
                 }
                 else {
                     this.cache.add(tile);
-                    if (tile instanceof MarkTile) {
+                    if (tile instanceof MarkTile)
                         activeMarks.unshift(tile.mark);
-                        openMarks++;
-                    }
                 }
                 this.openWidget = false;
             },
@@ -3071,7 +3073,7 @@ class DocView {
         let { anchorNode, anchorOffset } = view.observer.selectionRange;
         if (!sel || !cursor.empty || !cursor.assoc || !sel.modify)
             return;
-        let line = this.lineAt(cursor.head);
+        let line = this.lineAt(cursor.head, cursor.assoc);
         if (!line)
             return;
         let lineStart = line.posAtStart;
@@ -3150,7 +3152,7 @@ class DocView {
         let after, afterOff = -1, afterBad = false;
         this.tile.blockTiles((tile, off) => {
             if (tile.isWidget()) {
-                if ((tile.flags & 32 /* TileFlag.After */) && before)
+                if ((tile.flags & 32 /* TileFlag.After */) && off >= pos)
                     return true;
                 if (tile.flags & 16 /* TileFlag.Before */)
                     beforeBad = true;
@@ -3188,8 +3190,8 @@ class DocView {
         }
         return tile.coordsIn(offset, side);
     }
-    lineAt(pos) {
-        let { tile } = this.tile.resolveBlock(pos, 1);
+    lineAt(pos, side) {
+        let { tile } = this.tile.resolveBlock(pos, side);
         return tile.isLine() ? tile : null;
     }
     coordsForChar(pos) {
@@ -3735,7 +3737,10 @@ function posAtCoords(view, coords, precise, scanY) {
     if (block.type != BlockType.Text)
         return yOffset < (block.top + block.bottom) / 2 ? new PosAssoc(block.from, 1) : new PosAssoc(block.to, -1);
     // Here we know we're in a line, so run the logic for inline layout
-    return posAtCoordsInline(view, view.docView.lineAt(block.from), block.from, x, y);
+    let line = view.docView.lineAt(block.from, 2);
+    if (!line || line.length != block.length)
+        line = view.docView.lineAt(block.from, -2);
+    return posAtCoordsInline(view, line, block.from, x, y);
 }
 // Scan through the rectangles for the content of a tile, finding the
 // one closest to the given coordinates, prefering closeness in Y over
@@ -4654,7 +4659,8 @@ function eventBelongsToEditor(view, event) {
     if (event.defaultPrevented)
         return false;
     for (let node = event.target, tile; node != view.contentDOM; node = node.parentNode)
-        if (!node || node.nodeType == 11 || ((tile = Tile.get(node)) && tile.isWidget() && tile.widget.ignoreEvent(event)))
+        if (!node || node.nodeType == 11 ||
+            ((tile = Tile.get(node)) && tile.isWidget() && !tile.isHidden && tile.widget.ignoreEvent(event)))
             return false;
     return true;
 }
@@ -4773,42 +4779,13 @@ function rangeForClick(view, pos, bias, type) {
         return groupAt(view.state, pos, bias);
     }
     else { // Triple click
-        let visual = view.docView.lineAt(pos), line = view.state.doc.lineAt(visual ? visual.posAtEnd : pos);
+        let visual = view.docView.lineAt(pos, bias), line = view.state.doc.lineAt(visual ? visual.posAtEnd : pos);
         let from = visual ? visual.posAtStart : line.from, to = visual ? visual.posAtEnd : line.to;
         if (to < view.state.doc.length && to == line.to)
             to++;
         return EditorSelection.range(from, to);
     }
 }
-let inside = (x, y, rect) => y >= rect.top && y <= rect.bottom && x >= rect.left && x <= rect.right;
-// Try to determine, for the given coordinates, associated with the
-// given position, whether they are related to the element before or
-// the element after the position.
-function findPositionSide(view, pos, x, y) {
-    let line = view.docView.lineAt(pos);
-    if (!line)
-        return 1;
-    let off = pos - line.posAtStart;
-    // Line boundaries point into the line
-    if (off == 0)
-        return 1;
-    if (off == line.length)
-        return -1;
-    // Positions on top of an element point at that element
-    let before = line.coordsIn(off, -1);
-    if (before && inside(x, y, before))
-        return -1;
-    let after = line.coordsIn(off, 1);
-    if (after && inside(x, y, after))
-        return 1;
-    // This is probably a line wrap point. Pick before if the point is
-    // above its bottom.
-    return before && before.bottom >= y ? -1 : 1;
-}
-function queryPos(view, event) {
-    let pos = view.posAtCoords({ x: event.clientX, y: event.clientY }, false);
-    return { pos, bias: findPositionSide(view, pos, event.clientX, event.clientY) };
-}
 const BadMouseDetail = browser.ie && browser.ie_version <= 11;
 let lastMouseDown = null, lastMouseDownCount = 0, lastMouseDownTime = 0;
 function getClickType(event) {
@@ -4821,7 +4798,7 @@ function getClickType(event) {
         Math.abs(last.clientY - event.clientY) < 2) ? (lastMouseDownCount + 1) % 3 : 1;
 }
 function basicMouseSelection(view, event) {
-    let start = queryPos(view, event), type = getClickType(event);
+    let start = view.posAndSideAtCoords({ x: event.clientX, y: event.clientY }, false), type = getClickType(event);
     let startSel = view.state.selection;
     return {
         update(update) {
@@ -4831,10 +4808,10 @@ function basicMouseSelection(view, event) {
             }
         },
         get(event, extend, multiple) {
-            let cur = queryPos(view, event), removed;
-            let range = rangeForClick(view, cur.pos, cur.bias, type);
+            let cur = view.posAndSideAtCoords({ x: event.clientX, y: event.clientY }, false), removed;
+            let range = rangeForClick(view, cur.pos, cur.assoc, type);
             if (start.pos != cur.pos && !extend) {
-                let startRange = rangeForClick(view, start.pos, start.bias, type);
+                let startRange = rangeForClick(view, start.pos, start.assoc, type);
                 let from = Math.min(startRange.from, range.from), to = Math.max(startRange.to, range.to);
                 range = from < range.from ? EditorSelection.range(from, to) : EditorSelection.range(to, from);
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codemirror/view",
-  "version": "6.39.2",
+  "version": "6.39.4",
   "description": "DOM view component for the CodeMirror code editor",
   "scripts": {
     "test": "cm-runtests",