@codemirror/view 6.39.0-beta.5 → 6.39.1

package/CHANGELOG.md

@@ -1,3 +1,23 @@
+## 6.39.1 (2025-12-09)
+
+### Bug fixes
+
+Restore a workaround for a Chrome selection bug that had regressed in the previous release.
+
+## 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

@@ -1729,7 +1729,7 @@ class Tile {
     isText() { return false; }
     isBlock() { return false; }
     get domAttrs() { return null; }
-    sync() {
+    sync(track) {
         this.flags |= 2 /* TileFlag.Synced */;
         if (this.flags & 4 /* TileFlag.AttrsDirty */) {
             this.flags &= ~4 /* TileFlag.AttrsDirty */;
@@ -1801,16 +1801,19 @@ class CompositeTile extends Tile {
         this.children.push(child);
         child.parent = this;
     }
-    sync() {
+    sync(track) {
         if (this.flags & 2 /* TileFlag.Synced */)
             return;
-        super.sync();
+        super.sync(track);
         let parent = this.dom, prev = null, next;
+        let tracking = (track === null || track === void 0 ? void 0 : track.node) == parent ? track : null;
         let length = 0;
         for (let child of this.children) {
-            child.sync();
+            child.sync(track);
             length += child.length + child.breakAfter;
             next = prev ? prev.nextSibling : parent.firstChild;
+            if (tracking && next != child.dom)
+                tracking.written = true;
             if (child.dom.parentNode == parent) {
                 while (next && next != child.dom)
                     next = rm$1(next);
@@ -1821,6 +1824,8 @@ class CompositeTile extends Tile {
             prev = child.dom;
         }
         next = prev ? prev.nextSibling : parent.firstChild;
+        if (tracking && next)
+            tracking.written = true;
         while (next)
             next = rm$1(next);
         this.length = length;
@@ -2029,12 +2034,15 @@ class TextTile extends Tile {
         super(dom, text.length);
         this.text = text;
     }
-    sync() {
+    sync(track) {
         if (this.flags & 2 /* TileFlag.Synced */)
             return;
-        super.sync();
-        if (this.dom.nodeValue != this.text)
+        super.sync(track);
+        if (this.dom.nodeValue != this.text) {
+            if (track && track.node == this.dom)
+                track.written = true;
             this.dom.nodeValue = this.text;
+        }
     }
     isText() { return true; }
     toString() { return JSON.stringify(this.text); }
@@ -2939,8 +2947,8 @@ class DocView {
             // selection from the one it displays (issue #218). This tries
             // to detect that situation.
             let track = browser.chrome || browser.ios ? { node: observer.selectionRange.focusNode, written: false } : undefined;
-            this.tile.sync();
-            if (track && (track.written || observer.selectionRange.focusNode != track.node))
+            this.tile.sync(track);
+            if (track && (track.written || observer.selectionRange.focusNode != track.node || !this.tile.dom.contains(track.node)))
                 this.forceSelection = true;
             this.tile.dom.style.height = "";
         });
@@ -3303,8 +3311,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

@@ -1725,7 +1725,7 @@ class Tile {
     isText() { return false; }
     isBlock() { return false; }
     get domAttrs() { return null; }
-    sync() {
+    sync(track) {
         this.flags |= 2 /* TileFlag.Synced */;
         if (this.flags & 4 /* TileFlag.AttrsDirty */) {
             this.flags &= ~4 /* TileFlag.AttrsDirty */;
@@ -1797,16 +1797,19 @@ class CompositeTile extends Tile {
         this.children.push(child);
         child.parent = this;
     }
-    sync() {
+    sync(track) {
         if (this.flags & 2 /* TileFlag.Synced */)
             return;
-        super.sync();
+        super.sync(track);
         let parent = this.dom, prev = null, next;
+        let tracking = (track === null || track === void 0 ? void 0 : track.node) == parent ? track : null;
         let length = 0;
         for (let child of this.children) {
-            child.sync();
+            child.sync(track);
             length += child.length + child.breakAfter;
             next = prev ? prev.nextSibling : parent.firstChild;
+            if (tracking && next != child.dom)
+                tracking.written = true;
             if (child.dom.parentNode == parent) {
                 while (next && next != child.dom)
                     next = rm$1(next);
@@ -1817,6 +1820,8 @@ class CompositeTile extends Tile {
             prev = child.dom;
         }
         next = prev ? prev.nextSibling : parent.firstChild;
+        if (tracking && next)
+            tracking.written = true;
         while (next)
             next = rm$1(next);
         this.length = length;
@@ -2025,12 +2030,15 @@ class TextTile extends Tile {
         super(dom, text.length);
         this.text = text;
     }
-    sync() {
+    sync(track) {
         if (this.flags & 2 /* TileFlag.Synced */)
             return;
-        super.sync();
-        if (this.dom.nodeValue != this.text)
+        super.sync(track);
+        if (this.dom.nodeValue != this.text) {
+            if (track && track.node == this.dom)
+                track.written = true;
             this.dom.nodeValue = this.text;
+        }
     }
     isText() { return true; }
     toString() { return JSON.stringify(this.text); }
@@ -2935,8 +2943,8 @@ class DocView {
             // selection from the one it displays (issue #218). This tries
             // to detect that situation.
             let track = browser.chrome || browser.ios ? { node: observer.selectionRange.focusNode, written: false } : undefined;
-            this.tile.sync();
-            if (track && (track.written || observer.selectionRange.focusNode != track.node))
+            this.tile.sync(track);
+            if (track && (track.written || observer.selectionRange.focusNode != track.node || !this.tile.dom.contains(track.node)))
                 this.forceSelection = true;
             this.tile.dom.style.height = "";
         });
@@ -3299,8 +3307,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.5",
+  "version": "6.39.1",
   "description": "DOM view component for the CodeMirror code editor",
   "scripts": {
     "test": "cm-runtests",