@codemirror/view 6.43.7 → 6.43.9

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## 6.43.9 (2026-08-16)
+
+### Bug fixes
+
+Fix an issue where the scroll position would incorrectly be moved up when a document scrolled to the bottom would lose height.
+
+## 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

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;
@@ -3462,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) {
@@ -3888,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;
@@ -3929,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.
@@ -8229,10 +8236,12 @@ class EditorView {
                                 (scroll == this.scrollDOM || this.hasFocus ||
                                     Math.max(this.inputState.lastWheelEvent, this.inputState.lastTouchTime) > Date.now() - 100)) {
                                 scrollOffset = scrollOffset + diff;
-                                if (scroll)
-                                    scroll.scrollTop += diff;
-                                else
+                                if (!scroll)
                                     this.win.scrollBy(0, diff);
+                                else if (scrollAnchorPos < 0)
+                                    scroll.scrollTop = scroll.scrollHeight;
+                                else
+                                    scroll.scrollTop += diff;
                                 scrollAnchorHeight = -1;
                                 continue;
                             }
@@ -9597,7 +9606,8 @@ const selectionLayer = layer({
     },
     class: "cm-selectionLayer"
 });
-const selectionBg = browser.gecko && browser.gecko_version >= 153 ? "#ffffff01" : "transparent";
+// 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: `${selectionBg} !important` },

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;
@@ -3458,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) {
@@ -3884,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;
@@ -3925,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.
@@ -8224,10 +8231,12 @@ class EditorView {
                                 (scroll == this.scrollDOM || this.hasFocus ||
                                     Math.max(this.inputState.lastWheelEvent, this.inputState.lastTouchTime) > Date.now() - 100)) {
                                 scrollOffset = scrollOffset + diff;
-                                if (scroll)
-                                    scroll.scrollTop += diff;
-                                else
+                                if (!scroll)
                                     this.win.scrollBy(0, diff);
+                                else if (scrollAnchorPos < 0)
+                                    scroll.scrollTop = scroll.scrollHeight;
+                                else
+                                    scroll.scrollTop += diff;
                                 scrollAnchorHeight = -1;
                                 continue;
                             }
@@ -9592,7 +9601,8 @@ const selectionLayer = /*@__PURE__*/layer({
     },
     class: "cm-selectionLayer"
 });
-const selectionBg = browser.gecko && browser.gecko_version >= 153 ? "#ffffff01" : "transparent";
+// 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: `${selectionBg} !important` },

package/package.json

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