@codemirror/view 6.43.6 → 6.43.8

package/CHANGELOG.md

@@ -1,3 +1,23 @@
+## 6.43.8 (2026-08-04)
+
+### Bug fixes
+
+Fix an issue that could cause identical `posAtCoords` queries to return different results.
+
+Fix an issue where scrolling a position into view vertically on Android or iOS could cause horizontal scrolling motion.
+
+## 6.43.7 (2026-07-27)
+
+### Bug fixes
+
+Make the panel container top/bottom style a regular rule, rather than an inline style.
+
+Fix an issue where widgets changing length could break content updates.
+
+Fix incorrectly drawn selection when a line wrap point lies between widgets.
+
+Work around regression in Firefox 153 breaking selection hiding.
+
 ## 6.43.6 (2026-07-06)
 
 ### Bug fixes

package/dist/index.cjs

@@ -685,6 +685,22 @@ class DOMSelectionState {
         this.focusOffset = focusOffset;
     }
 }
+function getScrollStack(target) {
+    let stack = [];
+    for (let cur = target; cur; cur = cur.nodeType == 11 ? cur.host : cur.parentNode) {
+        if (cur.nodeType == 1)
+            stack.push({ node: cur, left: cur.scrollLeft, top: cur.scrollTop });
+    }
+    return stack;
+}
+function restoreScrollStack(stack, vert = true) {
+    for (let { node, left, top } of stack) {
+        if (vert && node.scrollTop != top)
+            node.scrollTop = top;
+        if (node.scrollLeft != left)
+            node.scrollLeft = left;
+    }
+}
 let preventScrollSupported = null;
 // Safari 26 breaks preventScroll support
 if (browser.safari && browser.safari_version >= 26)
@@ -696,12 +712,7 @@ function focusPreventScroll(dom) {
         return dom.setActive(); // in IE
     if (preventScrollSupported)
         return dom.focus(preventScrollSupported);
-    let stack = [];
-    for (let cur = dom; cur; cur = cur.parentNode) {
-        stack.push(cur, cur.scrollTop, cur.scrollLeft);
-        if (cur == cur.ownerDocument)
-            break;
-    }
+    let stack = getScrollStack(dom);
     dom.focus(preventScrollSupported == null ? {
         get preventScroll() {
             preventScrollSupported = { preventScroll: true };
@@ -710,13 +721,7 @@ function focusPreventScroll(dom) {
     } : undefined);
     if (!preventScrollSupported) {
         preventScrollSupported = false;
-        for (let i = 0; i < stack.length;) {
-            let elt = stack[i++], top = stack[i++], left = stack[i++];
-            if (elt.scrollTop != top)
-                elt.scrollTop = top;
-            if (elt.scrollLeft != left)
-                elt.scrollLeft = left;
-        }
+        restoreScrollStack(stack);
     }
 }
 let scratchRange;
@@ -1970,6 +1975,9 @@ class LineTile extends CompositeTile {
     }
     get domAttrs() { return this.attrs; }
     // Find the tile associated with a given position in this line.
+    // Side -2/2 is handled specially, in that it allows the position
+    // returned to be before (-2) or after (2) widgets that would always
+    // be after/before a cursor position.
     resolveInline(pos, side, forCoords) {
         let before = null, beforeOff = -1, after = null, afterOff = -1;
         function scan(tile, pos) {
@@ -1980,11 +1988,11 @@ class LineTile extends CompositeTile {
                         scan(child, pos - off);
                     }
                     else if ((!after || after.isHidden && (side > 0 && !(after.flags & 32 /* TileFlag.After */) || forCoords && onSameLine(after, child))) &&
-                        (end > pos || (child.flags & 32 /* TileFlag.After */))) {
+                        (end > pos || (child.flags & 32 /* TileFlag.After */) && side <= 1)) {
                         after = child;
                         afterOff = pos - off;
                     }
-                    else if (off < pos || (child.flags & 16 /* TileFlag.Before */) && !child.isHidden) {
+                    else if (off < pos || (child.flags & 16 /* TileFlag.Before */) && !child.isHidden && side >= -1) {
                         before = child;
                         beforeOff = pos - off;
                     }
@@ -2190,7 +2198,15 @@ class TilePointer {
         let { tile, index, beforeBreak, parents } = this;
         while (dist || side > 0) {
             if (!tile.isComposite()) {
-                if (index == tile.length) {
+                let len = tile.length;
+                if (index < len && dist) {
+                    let take = Math.min(dist, len - index);
+                    if (walker)
+                        walker.skip(tile, index, index + take);
+                    dist -= take;
+                    index += take;
+                }
+                if (index == len) {
                     beforeBreak = !!tile.breakAfter;
                     ({ tile, index } = parents.pop());
                     index++;
@@ -2198,13 +2214,6 @@ class TilePointer {
                 else if (!dist) {
                     break;
                 }
-                else {
-                    let take = Math.min(dist, tile.length - index);
-                    if (walker)
-                        walker.skip(tile, index, index + take);
-                    dist -= take;
-                    index += take;
-                }
             }
             else if (beforeBreak) {
                 if (!dist)
@@ -3458,11 +3467,14 @@ class DocView {
         // can affect it. So this tries to kludge around the problem by
         // calling scrollIntoView on the scroll target's line.
         if (window.visualViewport && window.innerHeight - window.visualViewport.height > 1 &&
-            (rect.top > window.pageYOffset + window.visualViewport.offsetTop + window.visualViewport.height ||
-                rect.bottom < window.pageYOffset + window.visualViewport.offsetTop)) {
+            (rect.top > window.visualViewport.offsetTop + window.visualViewport.height ||
+                rect.bottom < window.visualViewport.offsetTop)) {
             let line = this.view.docView.lineAt(range.head, 1);
-            if (line)
+            if (line) {
+                let stack = getScrollStack(line.dom);
                 line.dom.scrollIntoView({ block: "nearest" });
+                restoreScrollStack(stack, false);
+            }
         }
     }
     lineHasWidget(pos) {
@@ -3884,23 +3896,22 @@ class InlineCoordsScan {
         search: while (lo < hi) {
             let dist = hi - lo, mid = (lo + hi) >> 1;
             adjust: if (seen.has(mid)) {
-                let scan = lo + Math.floor(Math.random() * dist);
-                for (let i = 0; i < dist; i++) {
+                for (let i = 1; i < dist; i++) {
+                    let scan = mid + i;
+                    if (scan >= hi)
+                        scan -= dist;
                     if (!seen.has(scan)) {
                         mid = scan;
                         break adjust;
                     }
-                    scan++;
-                    if (scan == hi)
-                        scan = lo; // Wrap around
                 }
                 break search; // No index found, we're done
             }
             seen.add(mid);
-            let rects = getRects(mid);
+            let rects = getRects(mid), side = 0;
             if (rects)
                 for (let i = 0; i < rects.length; i++) {
-                    let rect = rects[i], side = 0;
+                    let rect = rects[i];
                     // Ignore empty rectangles when there are other rectangles
                     if (rect.width == 0 && rects.length > 1)
                         continue;
@@ -3925,12 +3936,12 @@ class InlineCoordsScan {
                         if (off)
                             side = (off < 0) == (this.baseDir == exports.Direction.LTR) ? -1 : 1;
                     }
-                    // Narrow binary search when it is safe to do so
-                    if (side == -1 && (!bidi || this.baseDirAt(positions[mid], 1)))
-                        hi = mid;
-                    else if (side == 1 && (!bidi || this.baseDirAt(positions[mid + 1], -1)))
-                        lo = mid + 1;
                 }
+            // Narrow binary search when it is safe to do so
+            if (side == -1 && (!bidi || this.baseDirAt(positions[mid], 1)))
+                hi = mid;
+            else if (side == 1 && (!bidi || this.baseDirAt(positions[mid + 1], -1)))
+                lo = mid + 1;
         }
         // If no element with y overlap is found, find the nearest element
         // on the y axis, move this.y into it, and retry the scan.
@@ -6983,6 +6994,8 @@ const baseTheme$1 = buildTheme("." + baseThemeID, {
         backgroundColor: "#f5f5f5",
         color: "black"
     },
+    ".cm-panels-top": { top: "0" },
+    ".cm-panels-bottom": { bottom: "0" },
     "&light .cm-panels-top": {
         borderBottom: "1px solid #ddd"
     },
@@ -9591,9 +9604,11 @@ const selectionLayer = layer({
     },
     class: "cm-selectionLayer"
 });
+// https://discuss.codemirror.net/t/firefox-153-ignores-transparent-selection-styling/9838
+const selectionBg = browser.gecko && browser.gecko_version == 153 ? "#ffffff01" : "transparent";
 const hideNativeSelection = state.Prec.highest(EditorView.theme({
     ".cm-line": {
-        "& ::selection, &::selection": { backgroundColor: "transparent !important" },
+        "& ::selection, &::selection": { backgroundColor: `${selectionBg} !important` },
         caretColor: "transparent !important"
     },
     ".cm-content": {
@@ -11104,7 +11119,6 @@ class PanelGroup {
         if (!this.dom) {
             this.dom = document.createElement("div");
             this.dom.className = this.top ? "cm-panels cm-panels-top" : "cm-panels cm-panels-bottom";
-            this.dom.style[this.top ? "top" : "bottom"] = "0";
             let parent = this.container || this.view.dom;
             parent.insertBefore(this.dom, this.top ? parent.firstChild : null);
         }

package/dist/index.js

@@ -682,6 +682,22 @@ class DOMSelectionState {
         this.focusOffset = focusOffset;
     }
 }
+function getScrollStack(target) {
+    let stack = [];
+    for (let cur = target; cur; cur = cur.nodeType == 11 ? cur.host : cur.parentNode) {
+        if (cur.nodeType == 1)
+            stack.push({ node: cur, left: cur.scrollLeft, top: cur.scrollTop });
+    }
+    return stack;
+}
+function restoreScrollStack(stack, vert = true) {
+    for (let { node, left, top } of stack) {
+        if (vert && node.scrollTop != top)
+            node.scrollTop = top;
+        if (node.scrollLeft != left)
+            node.scrollLeft = left;
+    }
+}
 let preventScrollSupported = null;
 // Safari 26 breaks preventScroll support
 if (browser.safari && browser.safari_version >= 26)
@@ -693,12 +709,7 @@ function focusPreventScroll(dom) {
         return dom.setActive(); // in IE
     if (preventScrollSupported)
         return dom.focus(preventScrollSupported);
-    let stack = [];
-    for (let cur = dom; cur; cur = cur.parentNode) {
-        stack.push(cur, cur.scrollTop, cur.scrollLeft);
-        if (cur == cur.ownerDocument)
-            break;
-    }
+    let stack = getScrollStack(dom);
     dom.focus(preventScrollSupported == null ? {
         get preventScroll() {
             preventScrollSupported = { preventScroll: true };
@@ -707,13 +718,7 @@ function focusPreventScroll(dom) {
     } : undefined);
     if (!preventScrollSupported) {
         preventScrollSupported = false;
-        for (let i = 0; i < stack.length;) {
-            let elt = stack[i++], top = stack[i++], left = stack[i++];
-            if (elt.scrollTop != top)
-                elt.scrollTop = top;
-            if (elt.scrollLeft != left)
-                elt.scrollLeft = left;
-        }
+        restoreScrollStack(stack);
     }
 }
 let scratchRange;
@@ -1966,6 +1971,9 @@ class LineTile extends CompositeTile {
     }
     get domAttrs() { return this.attrs; }
     // Find the tile associated with a given position in this line.
+    // Side -2/2 is handled specially, in that it allows the position
+    // returned to be before (-2) or after (2) widgets that would always
+    // be after/before a cursor position.
     resolveInline(pos, side, forCoords) {
         let before = null, beforeOff = -1, after = null, afterOff = -1;
         function scan(tile, pos) {
@@ -1976,11 +1984,11 @@ class LineTile extends CompositeTile {
                         scan(child, pos - off);
                     }
                     else if ((!after || after.isHidden && (side > 0 && !(after.flags & 32 /* TileFlag.After */) || forCoords && onSameLine(after, child))) &&
-                        (end > pos || (child.flags & 32 /* TileFlag.After */))) {
+                        (end > pos || (child.flags & 32 /* TileFlag.After */) && side <= 1)) {
                         after = child;
                         afterOff = pos - off;
                     }
-                    else if (off < pos || (child.flags & 16 /* TileFlag.Before */) && !child.isHidden) {
+                    else if (off < pos || (child.flags & 16 /* TileFlag.Before */) && !child.isHidden && side >= -1) {
                         before = child;
                         beforeOff = pos - off;
                     }
@@ -2186,7 +2194,15 @@ class TilePointer {
         let { tile, index, beforeBreak, parents } = this;
         while (dist || side > 0) {
             if (!tile.isComposite()) {
-                if (index == tile.length) {
+                let len = tile.length;
+                if (index < len && dist) {
+                    let take = Math.min(dist, len - index);
+                    if (walker)
+                        walker.skip(tile, index, index + take);
+                    dist -= take;
+                    index += take;
+                }
+                if (index == len) {
                     beforeBreak = !!tile.breakAfter;
                     ({ tile, index } = parents.pop());
                     index++;
@@ -2194,13 +2210,6 @@ class TilePointer {
                 else if (!dist) {
                     break;
                 }
-                else {
-                    let take = Math.min(dist, tile.length - index);
-                    if (walker)
-                        walker.skip(tile, index, index + take);
-                    dist -= take;
-                    index += take;
-                }
             }
             else if (beforeBreak) {
                 if (!dist)
@@ -3454,11 +3463,14 @@ class DocView {
         // can affect it. So this tries to kludge around the problem by
         // calling scrollIntoView on the scroll target's line.
         if (window.visualViewport && window.innerHeight - window.visualViewport.height > 1 &&
-            (rect.top > window.pageYOffset + window.visualViewport.offsetTop + window.visualViewport.height ||
-                rect.bottom < window.pageYOffset + window.visualViewport.offsetTop)) {
+            (rect.top > window.visualViewport.offsetTop + window.visualViewport.height ||
+                rect.bottom < window.visualViewport.offsetTop)) {
             let line = this.view.docView.lineAt(range.head, 1);
-            if (line)
+            if (line) {
+                let stack = getScrollStack(line.dom);
                 line.dom.scrollIntoView({ block: "nearest" });
+                restoreScrollStack(stack, false);
+            }
         }
     }
     lineHasWidget(pos) {
@@ -3880,23 +3892,22 @@ class InlineCoordsScan {
         search: while (lo < hi) {
             let dist = hi - lo, mid = (lo + hi) >> 1;
             adjust: if (seen.has(mid)) {
-                let scan = lo + Math.floor(Math.random() * dist);
-                for (let i = 0; i < dist; i++) {
+                for (let i = 1; i < dist; i++) {
+                    let scan = mid + i;
+                    if (scan >= hi)
+                        scan -= dist;
                     if (!seen.has(scan)) {
                         mid = scan;
                         break adjust;
                     }
-                    scan++;
-                    if (scan == hi)
-                        scan = lo; // Wrap around
                 }
                 break search; // No index found, we're done
             }
             seen.add(mid);
-            let rects = getRects(mid);
+            let rects = getRects(mid), side = 0;
             if (rects)
                 for (let i = 0; i < rects.length; i++) {
-                    let rect = rects[i], side = 0;
+                    let rect = rects[i];
                     // Ignore empty rectangles when there are other rectangles
                     if (rect.width == 0 && rects.length > 1)
                         continue;
@@ -3921,12 +3932,12 @@ class InlineCoordsScan {
                         if (off)
                             side = (off < 0) == (this.baseDir == Direction.LTR) ? -1 : 1;
                     }
-                    // Narrow binary search when it is safe to do so
-                    if (side == -1 && (!bidi || this.baseDirAt(positions[mid], 1)))
-                        hi = mid;
-                    else if (side == 1 && (!bidi || this.baseDirAt(positions[mid + 1], -1)))
-                        lo = mid + 1;
                 }
+            // Narrow binary search when it is safe to do so
+            if (side == -1 && (!bidi || this.baseDirAt(positions[mid], 1)))
+                hi = mid;
+            else if (side == 1 && (!bidi || this.baseDirAt(positions[mid + 1], -1)))
+                lo = mid + 1;
         }
         // If no element with y overlap is found, find the nearest element
         // on the y axis, move this.y into it, and retry the scan.
@@ -6978,6 +6989,8 @@ const baseTheme$1 = /*@__PURE__*/buildTheme("." + baseThemeID, {
         backgroundColor: "#f5f5f5",
         color: "black"
     },
+    ".cm-panels-top": { top: "0" },
+    ".cm-panels-bottom": { bottom: "0" },
     "&light .cm-panels-top": {
         borderBottom: "1px solid #ddd"
     },
@@ -9586,9 +9599,11 @@ const selectionLayer = /*@__PURE__*/layer({
     },
     class: "cm-selectionLayer"
 });
+// https://discuss.codemirror.net/t/firefox-153-ignores-transparent-selection-styling/9838
+const selectionBg = browser.gecko && browser.gecko_version == 153 ? "#ffffff01" : "transparent";
 const hideNativeSelection = /*@__PURE__*/Prec.highest(/*@__PURE__*/EditorView.theme({
     ".cm-line": {
-        "& ::selection, &::selection": { backgroundColor: "transparent !important" },
+        "& ::selection, &::selection": { backgroundColor: `${selectionBg} !important` },
         caretColor: "transparent !important"
     },
     ".cm-content": {
@@ -11099,7 +11114,6 @@ class PanelGroup {
         if (!this.dom) {
             this.dom = document.createElement("div");
             this.dom.className = this.top ? "cm-panels cm-panels-top" : "cm-panels cm-panels-bottom";
-            this.dom.style[this.top ? "top" : "bottom"] = "0";
             let parent = this.container || this.view.dom;
             parent.insertBefore(this.dom, this.top ? parent.firstChild : null);
         }

package/package.json

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