@codemirror/view 6.39.0-beta.4 → 6.39.0

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## 6.39.0 (2025-12-08)
+
+### Bug fixes
+
+Properly handle bidirectional text in `posAtCoords`.
+
+Avoid computing a zero character width (leading to divisions by zero) when the editor is hidden and the browser doesn't have a layout for it.
+
+### New features
+
+The `posAndSideAtCoords` method is an extended version of `posAtCoords` that also tells you which side of the position the coordinates are associated with.
+
+Add support for block wrappers, decoration-like things that allow extension code to create DOM nodes around groups of lines.
+
 ## 6.38.8 (2025-11-17)
 
 ### Bug fixes

package/dist/index.cjs

@@ -2397,7 +2397,7 @@ class TileBuilder {
         for (let cur = this.blockWrappers; cur.value && cur.from <= this.pos; cur.next())
             if (cur.to >= this.pos) {
                 let wrap = new OpenWrapper(cur.from, cur.to, cur.value, cur.rank), i = this.wrappers.length;
-                while (i > 0 && this.wrappers[i - 1].rank < wrap.rank)
+                while (i > 0 && (this.wrappers[i - 1].rank - wrap.rank || this.wrappers[i - 1].to - wrap.to) < 0)
                     i--;
                 this.wrappers.splice(i, 0, wrap);
             }
@@ -2653,8 +2653,10 @@ class TileUpdate {
                 }
                 else {
                     this.cache.add(tile);
-                    if (tile instanceof MarkTile)
+                    if (tile instanceof MarkTile) {
                         activeMarks.unshift(tile.mark);
+                        openMarks++;
+                    }
                 }
                 this.openWidget = false;
             },
@@ -3229,20 +3231,18 @@ class DocView {
                 if (pos > to)
                     break;
                 let child = tile.children[i], end = pos + child.length;
+                let childRect = child.dom.getBoundingClientRect(), { height } = childRect;
+                if (measureBounds && !i)
+                    spaceAbove += childRect.top - measureBounds.top;
                 if (child instanceof BlockWrapperTile) {
                     if (end > from)
-                        scan(child, pos, child.dom.getBoundingClientRect());
+                        scan(child, pos, childRect);
                 }
                 else if (pos >= from) {
-                    let childRect = child.dom.getBoundingClientRect(), { height } = childRect;
-                    if (measureBounds && !i)
-                        spaceAbove += childRect.top - measureBounds.top;
                     if (spaceAbove > 0)
                         result.push(-spaceAbove);
                     result.push(height + spaceAbove);
                     spaceAbove = 0;
-                    if (measureBounds && i == tile.children.length - 1)
-                        spaceAbove += measureBounds.bottom - childRect.bottom;
                     if (isWider) {
                         let last = child.dom.lastChild;
                         let rects = last ? clientRectsFor(last) : [];
@@ -3258,6 +3258,8 @@ class DocView {
                         }
                     }
                 }
+                if (measureBounds && i == tile.children.length - 1)
+                    spaceAbove += measureBounds.bottom - childRect.bottom;
                 pos = end + child.breakAfter;
             }
         };
@@ -3301,8 +3303,8 @@ class DocView {
             this.tile.dom.appendChild(dummy);
             let rect = clientRectsFor(dummy.firstChild)[0];
             lineHeight = dummy.getBoundingClientRect().height;
-            charWidth = rect ? rect.width / 27 : 7;
-            textHeight = rect ? rect.height : lineHeight;
+            charWidth = rect && rect.width ? rect.width / 27 : 7;
+            textHeight = rect && rect.height ? rect.height : lineHeight;
             dummy.remove();
         });
         return { lineHeight, charWidth, textHeight };

package/dist/index.js

@@ -2393,7 +2393,7 @@ class TileBuilder {
         for (let cur = this.blockWrappers; cur.value && cur.from <= this.pos; cur.next())
             if (cur.to >= this.pos) {
                 let wrap = new OpenWrapper(cur.from, cur.to, cur.value, cur.rank), i = this.wrappers.length;
-                while (i > 0 && this.wrappers[i - 1].rank < wrap.rank)
+                while (i > 0 && (this.wrappers[i - 1].rank - wrap.rank || this.wrappers[i - 1].to - wrap.to) < 0)
                     i--;
                 this.wrappers.splice(i, 0, wrap);
             }
@@ -2649,8 +2649,10 @@ class TileUpdate {
                 }
                 else {
                     this.cache.add(tile);
-                    if (tile instanceof MarkTile)
+                    if (tile instanceof MarkTile) {
                         activeMarks.unshift(tile.mark);
+                        openMarks++;
+                    }
                 }
                 this.openWidget = false;
             },
@@ -3225,20 +3227,18 @@ class DocView {
                 if (pos > to)
                     break;
                 let child = tile.children[i], end = pos + child.length;
+                let childRect = child.dom.getBoundingClientRect(), { height } = childRect;
+                if (measureBounds && !i)
+                    spaceAbove += childRect.top - measureBounds.top;
                 if (child instanceof BlockWrapperTile) {
                     if (end > from)
-                        scan(child, pos, child.dom.getBoundingClientRect());
+                        scan(child, pos, childRect);
                 }
                 else if (pos >= from) {
-                    let childRect = child.dom.getBoundingClientRect(), { height } = childRect;
-                    if (measureBounds && !i)
-                        spaceAbove += childRect.top - measureBounds.top;
                     if (spaceAbove > 0)
                         result.push(-spaceAbove);
                     result.push(height + spaceAbove);
                     spaceAbove = 0;
-                    if (measureBounds && i == tile.children.length - 1)
-                        spaceAbove += measureBounds.bottom - childRect.bottom;
                     if (isWider) {
                         let last = child.dom.lastChild;
                         let rects = last ? clientRectsFor(last) : [];
@@ -3254,6 +3254,8 @@ class DocView {
                         }
                     }
                 }
+                if (measureBounds && i == tile.children.length - 1)
+                    spaceAbove += measureBounds.bottom - childRect.bottom;
                 pos = end + child.breakAfter;
             }
         };
@@ -3297,8 +3299,8 @@ class DocView {
             this.tile.dom.appendChild(dummy);
             let rect = clientRectsFor(dummy.firstChild)[0];
             lineHeight = dummy.getBoundingClientRect().height;
-            charWidth = rect ? rect.width / 27 : 7;
-            textHeight = rect ? rect.height : lineHeight;
+            charWidth = rect && rect.width ? rect.width / 27 : 7;
+            textHeight = rect && rect.height ? rect.height : lineHeight;
             dummy.remove();
         });
         return { lineHeight, charWidth, textHeight };

package/package.json

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