@codemirror/view 6.39.7 → 6.39.9

package/CHANGELOG.md

@@ -1,3 +1,23 @@
+## 6.39.9 (2026-01-06)
+
+### Bug fixes
+
+Fix a bug where `EditorSelection.cursor()` with a non-zero `assoc` value would not be visually respected at soft-wrap boundaries on initial view creation.
+
+Fix error caused by hover tooltips running a scheduled timeout after their editor has been destroyed.
+
+Fix a bug that caused `EditorView.outerDecorations` to not affect the content height map.
+
+Fix an issue where composition near a widget could get unnecessarily interrupted.
+
+## 6.39.8 (2025-12-30)
+
+### Bug fixes
+
+Fix a bug that cause `coordsAtPos` to use the dimensions of widget buffers when there were more meaningful elements to use nearby.
+
+Fix a data structure corruption that could cause crashes during viewport changes.
+
 ## 6.39.7 (2025-12-24)
 
 ### Bug fixes

package/dist/index.cjs

@@ -2149,7 +2149,7 @@ class WidgetBufferTile extends Tile {
         img.setAttribute("aria-hidden", "true");
         super(img, 0, flags);
     }
-    get isHidden() { return false; }
+    get isHidden() { return true; }
     get overrideDOMText() { return state.Text.empty; }
     coordsIn(pos) { return this.dom.getBoundingClientRect(); }
 }
@@ -2502,6 +2502,8 @@ class TileCache {
     }
     // Put a tile in the cache.
     add(tile) {
+        if (tile.demo)
+            console.log("Add widget to cache");
         let i = tile.constructor.bucket, bucket = this.buckets[i];
         if (bucket.length < 6 /* C.Bucket */)
             bucket.push(tile);
@@ -2526,6 +2528,8 @@ class TileCache {
     }
     findWidget(widget, length, flags) {
         let widgets = this.buckets[0];
+        if (widget.demo)
+            console.log("looking for widget", widget, "in cache", widgets.slice());
         if (widgets.length)
             for (let i = 0, pass = 0;; i++) {
                 if (i == widgets.length) {
@@ -2541,10 +2545,14 @@ class TileCache {
                     widgets.splice(i, 1);
                     if (i < this.index[0])
                         this.index[0]--;
-                    this.reused.set(tile, 1 /* Reused.Full */);
-                    tile.length = length;
-                    tile.flags = (tile.flags & ~(496 /* TileFlag.Widget */ | 1 /* TileFlag.BreakAfter */)) | flags;
-                    return tile;
+                    if (tile.length == length && (tile.flags & (496 /* TileFlag.Widget */ | 1 /* TileFlag.BreakAfter */)) == flags) {
+                        this.reused.set(tile, 1 /* Reused.Full */);
+                        return tile;
+                    }
+                    else {
+                        this.reused.set(tile, 2 /* Reused.DOM */);
+                        return new WidgetTile(tile.dom, length, widget, (tile.flags & ~(496 /* TileFlag.Widget */ | 1 /* TileFlag.BreakAfter */)) | flags);
+                    }
                 }
             }
     }
@@ -2558,6 +2566,10 @@ class TileCache {
         this.reused.set(tile, type);
         return tile.dom;
     }
+    clear() {
+        for (let i = 0; i < this.buckets.length; i++)
+            this.buckets[i].length = this.index[i] = 0;
+    }
 }
 // This class organizes a pass over the document, guided by the array
 // of replaced ranges. For ranges that haven't changed, it iterates
@@ -2600,17 +2612,20 @@ class TileUpdate {
             }
             if (!next)
                 break;
-            this.forward(next.fromA, next.toA);
             // Compositions need to be handled specially, forcing the
             // focused text node and its parent nodes to remain stable at
             // that point in the document.
             if (composition && next.fromA <= composition.range.fromA && next.toA >= composition.range.toA) {
+                this.forward(next.fromA, composition.range.fromA);
                 this.emit(posB, composition.range.fromB);
+                this.cache.clear(); // Must not reuse DOM across composition
                 this.builder.addComposition(composition, compositionContext);
                 this.text.skip(composition.range.toB - composition.range.fromB);
+                this.forward(composition.range.fromA, next.toA);
                 this.emit(composition.range.toB, next.toB);
             }
             else {
+                this.forward(next.fromA, next.toA);
                 this.emit(posB, next.toB);
             }
             posB = next.toB;
@@ -6023,7 +6038,7 @@ class ViewState {
         this.mustEnforceCursorAssoc = false;
         let guessWrapping = state$1.facet(contentAttributes).some(v => typeof v != "function" && v.class == "cm-lineWrapping");
         this.heightOracle = new HeightOracle(guessWrapping);
-        this.stateDeco = state$1.facet(decorations).filter(d => typeof d != "function");
+        this.stateDeco = staticDeco(state$1);
         this.heightMap = HeightMap.empty().applyChanges(this.stateDeco, state.Text.empty, this.heightOracle.setDoc(state$1.doc), [new ChangedRange(0, 0, 0, state$1.doc.length)]);
         for (let i = 0; i < 2; i++) {
             this.viewport = this.getViewport(0, null);
@@ -6062,7 +6077,7 @@ class ViewState {
     update(update, scrollTarget = null) {
         this.state = update.state;
         let prevDeco = this.stateDeco;
-        this.stateDeco = this.state.facet(decorations).filter(d => typeof d != "function");
+        this.stateDeco = staticDeco(this.state);
         let contentChanges = update.changedRanges;
         let heightChanges = ChangedRange.extendWithRanges(contentChanges, heightRelevantDecoChanges(prevDeco, this.stateDeco, update ? update.changes : state.ChangeSet.empty(this.state.doc.length)));
         let prevHeight = this.heightMap.height;
@@ -6093,7 +6108,7 @@ class ViewState {
         update.flags |= this.computeVisibleRanges(update.changes);
         if (scrollTarget)
             this.scrollTarget = scrollTarget;
-        if (!this.mustEnforceCursorAssoc && update.selectionSet && update.view.lineWrapping &&
+        if (!this.mustEnforceCursorAssoc && (update.selectionSet || update.focusChanged) && update.view.lineWrapping &&
             update.state.selection.main.empty && update.state.selection.main.assoc &&
             !update.state.facet(nativeSelectionHidden))
             this.mustEnforceCursorAssoc = true;
@@ -6491,6 +6506,13 @@ const IdScaler = {
     scale: 1,
     eq(other) { return other == this; }
 };
+function staticDeco(state$1) {
+    let deco = state$1.facet(decorations).filter(d => typeof d != "function");
+    let outer = state$1.facet(outerDecorations).filter(d => typeof d != "function");
+    if (outer.length)
+        deco.push(state.RangeSet.join(outer));
+    return deco;
+}
 // When the height is too big (> VP.MaxDOMHeight), scale down the
 // regions outside the viewports so that the total height is
 // VP.MaxDOMHeight.
@@ -10520,6 +10542,7 @@ class HoverPlugin {
     }
     destroy() {
         clearTimeout(this.hoverTimeout);
+        clearTimeout(this.restartTimeout);
         this.view.dom.removeEventListener("mouseleave", this.mouseleave);
         this.view.dom.removeEventListener("mousemove", this.mousemove);
     }

package/dist/index.js

@@ -2145,7 +2145,7 @@ class WidgetBufferTile extends Tile {
         img.setAttribute("aria-hidden", "true");
         super(img, 0, flags);
     }
-    get isHidden() { return false; }
+    get isHidden() { return true; }
     get overrideDOMText() { return Text.empty; }
     coordsIn(pos) { return this.dom.getBoundingClientRect(); }
 }
@@ -2498,6 +2498,8 @@ class TileCache {
     }
     // Put a tile in the cache.
     add(tile) {
+        if (tile.demo)
+            console.log("Add widget to cache");
         let i = tile.constructor.bucket, bucket = this.buckets[i];
         if (bucket.length < 6 /* C.Bucket */)
             bucket.push(tile);
@@ -2522,6 +2524,8 @@ class TileCache {
     }
     findWidget(widget, length, flags) {
         let widgets = this.buckets[0];
+        if (widget.demo)
+            console.log("looking for widget", widget, "in cache", widgets.slice());
         if (widgets.length)
             for (let i = 0, pass = 0;; i++) {
                 if (i == widgets.length) {
@@ -2537,10 +2541,14 @@ class TileCache {
                     widgets.splice(i, 1);
                     if (i < this.index[0])
                         this.index[0]--;
-                    this.reused.set(tile, 1 /* Reused.Full */);
-                    tile.length = length;
-                    tile.flags = (tile.flags & ~(496 /* TileFlag.Widget */ | 1 /* TileFlag.BreakAfter */)) | flags;
-                    return tile;
+                    if (tile.length == length && (tile.flags & (496 /* TileFlag.Widget */ | 1 /* TileFlag.BreakAfter */)) == flags) {
+                        this.reused.set(tile, 1 /* Reused.Full */);
+                        return tile;
+                    }
+                    else {
+                        this.reused.set(tile, 2 /* Reused.DOM */);
+                        return new WidgetTile(tile.dom, length, widget, (tile.flags & ~(496 /* TileFlag.Widget */ | 1 /* TileFlag.BreakAfter */)) | flags);
+                    }
                 }
             }
     }
@@ -2554,6 +2562,10 @@ class TileCache {
         this.reused.set(tile, type);
         return tile.dom;
     }
+    clear() {
+        for (let i = 0; i < this.buckets.length; i++)
+            this.buckets[i].length = this.index[i] = 0;
+    }
 }
 // This class organizes a pass over the document, guided by the array
 // of replaced ranges. For ranges that haven't changed, it iterates
@@ -2596,17 +2608,20 @@ class TileUpdate {
             }
             if (!next)
                 break;
-            this.forward(next.fromA, next.toA);
             // Compositions need to be handled specially, forcing the
             // focused text node and its parent nodes to remain stable at
             // that point in the document.
             if (composition && next.fromA <= composition.range.fromA && next.toA >= composition.range.toA) {
+                this.forward(next.fromA, composition.range.fromA);
                 this.emit(posB, composition.range.fromB);
+                this.cache.clear(); // Must not reuse DOM across composition
                 this.builder.addComposition(composition, compositionContext);
                 this.text.skip(composition.range.toB - composition.range.fromB);
+                this.forward(composition.range.fromA, next.toA);
                 this.emit(composition.range.toB, next.toB);
             }
             else {
+                this.forward(next.fromA, next.toA);
                 this.emit(posB, next.toB);
             }
             posB = next.toB;
@@ -6018,7 +6033,7 @@ class ViewState {
         this.mustEnforceCursorAssoc = false;
         let guessWrapping = state.facet(contentAttributes).some(v => typeof v != "function" && v.class == "cm-lineWrapping");
         this.heightOracle = new HeightOracle(guessWrapping);
-        this.stateDeco = state.facet(decorations).filter(d => typeof d != "function");
+        this.stateDeco = staticDeco(state);
         this.heightMap = HeightMap.empty().applyChanges(this.stateDeco, Text.empty, this.heightOracle.setDoc(state.doc), [new ChangedRange(0, 0, 0, state.doc.length)]);
         for (let i = 0; i < 2; i++) {
             this.viewport = this.getViewport(0, null);
@@ -6057,7 +6072,7 @@ class ViewState {
     update(update, scrollTarget = null) {
         this.state = update.state;
         let prevDeco = this.stateDeco;
-        this.stateDeco = this.state.facet(decorations).filter(d => typeof d != "function");
+        this.stateDeco = staticDeco(this.state);
         let contentChanges = update.changedRanges;
         let heightChanges = ChangedRange.extendWithRanges(contentChanges, heightRelevantDecoChanges(prevDeco, this.stateDeco, update ? update.changes : ChangeSet.empty(this.state.doc.length)));
         let prevHeight = this.heightMap.height;
@@ -6088,7 +6103,7 @@ class ViewState {
         update.flags |= this.computeVisibleRanges(update.changes);
         if (scrollTarget)
             this.scrollTarget = scrollTarget;
-        if (!this.mustEnforceCursorAssoc && update.selectionSet && update.view.lineWrapping &&
+        if (!this.mustEnforceCursorAssoc && (update.selectionSet || update.focusChanged) && update.view.lineWrapping &&
             update.state.selection.main.empty && update.state.selection.main.assoc &&
             !update.state.facet(nativeSelectionHidden))
             this.mustEnforceCursorAssoc = true;
@@ -6486,6 +6501,13 @@ const IdScaler = {
     scale: 1,
     eq(other) { return other == this; }
 };
+function staticDeco(state) {
+    let deco = state.facet(decorations).filter(d => typeof d != "function");
+    let outer = state.facet(outerDecorations).filter(d => typeof d != "function");
+    if (outer.length)
+        deco.push(RangeSet.join(outer));
+    return deco;
+}
 // When the height is too big (> VP.MaxDOMHeight), scale down the
 // regions outside the viewports so that the total height is
 // VP.MaxDOMHeight.
@@ -10515,6 +10537,7 @@ class HoverPlugin {
     }
     destroy() {
         clearTimeout(this.hoverTimeout);
+        clearTimeout(this.restartTimeout);
         this.view.dom.removeEventListener("mouseleave", this.mouseleave);
         this.view.dom.removeEventListener("mousemove", this.mousemove);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codemirror/view",
-  "version": "6.39.7",
+  "version": "6.39.9",
   "description": "DOM view component for the CodeMirror code editor",
   "scripts": {
     "test": "cm-runtests",
@@ -36,6 +36,6 @@
   },
   "repository": {
     "type": "git",
-    "url": "https://github.com/codemirror/view.git"
+    "url": "git+https://github.com/codemirror/view.git"
   }
 }