@codemirror/view 0.19.35 → 0.19.39

package/CHANGELOG.md

@@ -1,3 +1,39 @@
+## 0.19.39 (2022-01-06)
+
+### Bug fixes
+
+Make sure the editor signals a `geometryChanged` update when its width changes.
+
+### New features
+
+`EditorView.darkTheme` can now be queried to figure out whether the editor is using a dark theme.
+
+## 0.19.38 (2022-01-05)
+
+### Bug fixes
+
+Fix a bug that caused line decorations with a `class` property to suppress all other line decorations for that line.
+
+Fix a bug that caused scroll effects to be corrupted when further updates came in before they were applied.
+
+Fix an issue where, depending on which way a floating point rounding error fell, `posAtCoords` (and thus vertical cursor motion) for positions outside of the vertical range of the document might or might not return the start/end of the document.
+
+## 0.19.37 (2021-12-22)
+
+### Bug fixes
+
+Fix regression where plugin replacing decorations that span to the end of the line are ignored.
+
+## 0.19.36 (2021-12-22)
+
+### Bug fixes
+
+Fix a crash in `posAtCoords` when the position lies in a block widget that is rendered but scrolled out of view.
+
+Adding block decorations from a plugin now raises an error. Replacing decorations that cross lines are ignored, when provided by a plugin.
+
+Fix inverted interpretation of the `precise` argument to `posAtCoords`.
+
 ## 0.19.35 (2021-12-20)
 
 ### Bug fixes

package/dist/index.cjs

@@ -1311,7 +1311,7 @@ class LineView extends ContentView {
         if (attrs)
             this.attrs = combineAttrs(attrs, this.attrs || {});
         if (cls)
-            this.attrs = combineAttrs(attrs, { class: cls });
+            this.attrs = combineAttrs({ class: cls }, this.attrs || {});
     }
     domAtPos(pos) {
         return inlineDOMAtPos(this.dom, this.children, pos);
@@ -1372,16 +1372,17 @@ class LineView extends ContentView {
     become(_other) { return false; }
     get type() { return exports.BlockType.Text; }
     static find(docView, pos) {
-        for (let i = 0, off = 0;; i++) {
+        for (let i = 0, off = 0; i < docView.children.length; i++) {
             let block = docView.children[i], end = off + block.length;
             if (end >= pos) {
                 if (block instanceof LineView)
                     return block;
-                if (block.length)
-                    return null;
+                if (end > pos)
+                    break;
             }
             off = end + block.breakAfter;
         }
+        return null;
     }
 }
 class BlockWidgetView extends ContentView {
@@ -1442,10 +1443,11 @@ class BlockWidgetView extends ContentView {
 }
 
 class ContentBuilder {
-    constructor(doc, pos, end) {
+    constructor(doc, pos, end, disallowBlockEffectsBelow) {
         this.doc = doc;
         this.pos = pos;
         this.end = end;
+        this.disallowBlockEffectsBelow = disallowBlockEffectsBelow;
         this.content = [];
         this.curLine = null;
         this.breakAtStart = 0;
@@ -1574,8 +1576,15 @@ class ContentBuilder {
         if (this.openStart < 0)
             this.openStart = openStart;
     }
-    static build(text, from, to, decorations) {
-        let builder = new ContentBuilder(text, from, to);
+    filterPoint(from, to, value, index) {
+        if (index >= this.disallowBlockEffectsBelow || !(value instanceof PointDecoration))
+            return true;
+        if (value.block)
+            throw new RangeError("Block decorations may not be specified via plugins");
+        return to <= this.doc.lineAt(this.pos).to;
+    }
+    static build(text, from, to, decorations, pluginDecorationLength) {
+        let builder = new ContentBuilder(text, from, to, pluginDecorationLength);
         builder.openEnd = rangeset.RangeSet.spans(decorations, from, to, builder);
         if (builder.openStart < 0)
             builder.openStart = builder.openEnd;
@@ -1698,11 +1707,13 @@ This field can be used by plugins to provide
 **Note**: For reasons of data flow (plugins are only updated
 after the viewport is computed), decorations produced by plugins
 are _not_ taken into account when predicting the vertical layout
-structure of the editor. Thus, things like large widgets or big
-replacements (i.e. code folding) should be provided through the
-state-level [`decorations` facet](https://codemirror.net/6/docs/ref/#view.EditorView^decorations),
-not this plugin field. Specifically, replacing decorations that
-cross line boundaries will break if provided through a plugin.
+structure of the editor. They **must not** introduce block
+widgets (that will raise an error) or replacing decorations that
+cover line breaks (these will be ignored if they occur). Such
+decorations, or others that cause a large amount of vertical
+size shift compared to the undecorated content, should be
+provided through the state-level [`decorations`
+facet](https://codemirror.net/6/docs/ref/#view.EditorView^decorations) instead.
 */
 PluginField.decorations = PluginField.define();
 /**
@@ -1935,8 +1946,6 @@ class ViewUpdate {
             view.inputState.notifiedFocused = focus;
             this.flags |= 1 /* Focus */;
         }
-        if (this.docChanged)
-            this.flags |= 2 /* Height */;
     }
     /**
     Tells you whether the [viewport](https://codemirror.net/6/docs/ref/#view.EditorView.viewport) or
@@ -1947,14 +1956,15 @@ class ViewUpdate {
         return (this.flags & 4 /* Viewport */) > 0;
     }
     /**
-    Indicates whether the line height in the editor changed in this update.
+    Indicates whether the height of an element in the editor changed
+    in this update.
     */
     get heightChanged() {
         return (this.flags & 2 /* Height */) > 0;
     }
     /**
-    Returns true when the document changed or the size of the editor
-    or the lines or characters within it has changed.
+    Returns true when the document was modified or the size of the
+    editor, or elements within the editor, changed.
     */
     get geometryChanged() {
         return this.docChanged || (this.flags & (8 /* Geometry */ | 2 /* Height */)) > 0;
@@ -2373,6 +2383,7 @@ class DocView extends ContentView {
         this.view = view;
         this.compositionDeco = Decoration.none;
         this.decorations = [];
+        this.pluginDecorationLength = 0;
         // Track a minimum width for the editor. When measuring sizes in
         // measureVisibleLineHeights, this is updated to point at the width
         // of a given element and its extent in the document. When a change
@@ -2394,7 +2405,8 @@ class DocView extends ContentView {
         this.setDOM(view.contentDOM);
         this.children = [new LineView];
         this.children[0].setParent(this);
-        this.updateInner([new ChangedRange(0, 0, 0, view.state.doc.length)], this.updateDeco(), 0);
+        this.updateDeco();
+        this.updateInner([new ChangedRange(0, 0, 0, view.state.doc.length)], 0);
     }
     get root() { return this.view.root; }
     get editorView() { return this.view; }
@@ -2433,7 +2445,7 @@ class DocView extends ContentView {
             return false;
         }
         else {
-            this.updateInner(changedRanges, deco, update.startState.doc.length);
+            this.updateInner(changedRanges, update.startState.doc.length);
             if (update.transactions.length)
                 this.lastUpdate = Date.now();
             return true;
@@ -2441,9 +2453,9 @@ class DocView extends ContentView {
     }
     // Used by update and the constructor do perform the actual DOM
     // update
-    updateInner(changes, deco, oldLength) {
+    updateInner(changes, oldLength) {
         this.view.viewState.mustMeasureContent = true;
-        this.updateChildren(changes, deco, oldLength);
+        this.updateChildren(changes, oldLength);
         let { observer } = this.view;
         observer.ignore(() => {
             // Lock the height during redrawing, since Chrome sometimes
@@ -2470,14 +2482,14 @@ class DocView extends ContentView {
                     gaps.push(child.dom);
         observer.updateGaps(gaps);
     }
-    updateChildren(changes, deco, oldLength) {
+    updateChildren(changes, oldLength) {
         let cursor = this.childCursor(oldLength);
         for (let i = changes.length - 1;; i--) {
             let next = i >= 0 ? changes[i] : null;
             if (!next)
                 break;
             let { fromA, toA, fromB, toB } = next;
-            let { content, breakAtStart, openStart, openEnd } = ContentBuilder.build(this.view.state.doc, fromB, toB, deco);
+            let { content, breakAtStart, openStart, openEnd } = ContentBuilder.build(this.view.state.doc, fromB, toB, this.decorations, this.pluginDecorationLength);
             let { i: toI, off: toOff } = cursor.findPos(toA, 1);
             let { i: fromI, off: fromOff } = cursor.findPos(fromA, -1);
             replaceRange(this, fromI, fromOff, toI, toOff, content, breakAtStart, openStart, openEnd);
@@ -2696,8 +2708,10 @@ class DocView extends ContentView {
         return Decoration.set(deco);
     }
     updateDeco() {
+        let pluginDecorations = this.view.pluginField(PluginField.decorations);
+        this.pluginDecorationLength = pluginDecorations.length;
         return this.decorations = [
-            ...this.view.pluginField(PluginField.decorations),
+            ...pluginDecorations,
             ...this.view.state.facet(decorations),
             this.compositionDeco,
             this.computeBlockGapDeco(),
@@ -2991,7 +3005,11 @@ function posAtCoords(view, { x, y }, precise, bias = -1) {
     var _a;
     let content = view.contentDOM.getBoundingClientRect(), docTop = content.top + view.viewState.paddingTop;
     let block, { docHeight } = view.viewState;
-    let yOffset = Math.max(0, Math.min(y - docTop, docHeight));
+    let yOffset = y - docTop;
+    if (yOffset < 0)
+        return 0;
+    if (yOffset > docHeight)
+        return view.state.doc.length;
     // Scan for a text block near the queried y position
     for (let halfLine = view.defaultLineHeight / 2, bounced = false;;) {
         block = view.elementAtHeight(yOffset);
@@ -3014,10 +3032,10 @@ function posAtCoords(view, { x, y }, precise, bias = -1) {
     let lineStart = block.from;
     // If this is outside of the rendered viewport, we can't determine a position
     if (lineStart < view.viewport.from)
-        return view.viewport.from == 0 ? 0 : precise ? posAtCoordsImprecise(view, content, block, x, y) : null;
+        return view.viewport.from == 0 ? 0 : precise ? null : posAtCoordsImprecise(view, content, block, x, y);
     if (lineStart > view.viewport.to)
         return view.viewport.to == view.state.doc.length ? view.state.doc.length :
-            precise ? posAtCoordsImprecise(view, content, block, x, y) : null;
+            precise ? null : posAtCoordsImprecise(view, content, block, x, y);
     // Prefer ShadowRootOrDocument.elementFromPoint if present, fall back to document if not
     let doc = view.dom.ownerDocument;
     let root = view.root.elementFromPoint ? view.root : doc;
@@ -3052,6 +3070,8 @@ function posAtCoords(view, { x, y }, precise, bias = -1) {
     // No luck, do our own (potentially expensive) search
     if (!node || !view.docView.dom.contains(node)) {
         let line = LineView.find(view.docView, lineStart);
+        if (!line)
+            return yOffset > block.top + block.height / 2 ? block.to : block.from;
         ({ node, offset } = domPosAtCoords(line.dom, x, y));
     }
     return view.docView.posFromDOM(node, offset);
@@ -4579,6 +4599,7 @@ class ViewState {
         this.contentDOMWidth = 0;
         this.contentDOMHeight = 0;
         this.editorHeight = 0;
+        this.editorWidth = 0;
         this.heightOracle = new HeightOracle;
         // See VP.MaxDOMHeight
         this.scaler = IdScaler;
@@ -4684,11 +4705,18 @@ class ViewState {
         }
         if (!this.inView)
             return 0;
+        let contentWidth = dom.clientWidth;
+        if (this.contentDOMWidth != contentWidth || this.editorHeight != view.scrollDOM.clientHeight ||
+            this.editorWidth != view.scrollDOM.clientWidth) {
+            this.contentDOMWidth = contentWidth;
+            this.editorHeight = view.scrollDOM.clientHeight;
+            this.editorWidth = view.scrollDOM.clientWidth;
+            result |= 8 /* Geometry */;
+        }
         if (measureContent) {
             let lineHeights = view.docView.measureVisibleLineHeights();
             if (oracle.mustRefreshForHeights(lineHeights))
                 refresh = true;
-            let contentWidth = dom.clientWidth;
             if (refresh || oracle.lineWrapping && Math.abs(contentWidth - this.contentDOMWidth) > oracle.charWidth) {
                 let { lineHeight, charWidth } = view.docView.measureTextSize();
                 refresh = oracle.refresh(whiteSpace, direction, lineHeight, charWidth, contentWidth / charWidth, lineHeights);
@@ -4697,14 +4725,6 @@ class ViewState {
                     result |= 8 /* Geometry */;
                 }
             }
-            if (this.contentDOMWidth != contentWidth) {
-                this.contentDOMWidth = contentWidth;
-                result |= 8 /* Geometry */;
-            }
-            if (this.editorHeight != view.scrollDOM.clientHeight) {
-                this.editorHeight = view.scrollDOM.clientHeight;
-                result |= 8 /* Geometry */;
-            }
             if (dTop > 0 && dBottom > 0)
                 bias = Math.max(dTop, dBottom);
             else if (dTop < 0 && dBottom < 0)
@@ -5844,7 +5864,7 @@ class EditorView {
         if (state$1.facet(state.EditorState.phrases) != this.state.facet(state.EditorState.phrases))
             return this.setState(state$1);
         update = new ViewUpdate(this, state$1, transactions);
-        let scrollTarget = null;
+        let scrollTarget = this.viewState.scrollTarget;
         try {
             this.updateState = 2 /* Updating */;
             for (let tr of transactions) {
@@ -6565,6 +6585,13 @@ mechanism for providing decorations.
 */
 EditorView.decorations = decorations;
 /**
+This facet records whether a dark theme is active. The extension
+returned by [`theme`](https://codemirror.net/6/docs/ref/#view.EditorView^theme) automatically
+includes an instance of this when the `dark` option is set to
+true.
+*/
+EditorView.darkTheme = darkTheme;
+/**
 Facet that provides additional DOM attributes for the editor's
 editable DOM element.
 */

package/dist/index.d.ts

@@ -340,11 +340,13 @@ declare class PluginField<T> {
     **Note**: For reasons of data flow (plugins are only updated
     after the viewport is computed), decorations produced by plugins
     are _not_ taken into account when predicting the vertical layout
-    structure of the editor. Thus, things like large widgets or big
-    replacements (i.e. code folding) should be provided through the
-    state-level [`decorations` facet](https://codemirror.net/6/docs/ref/#view.EditorView^decorations),
-    not this plugin field. Specifically, replacing decorations that
-    cross line boundaries will break if provided through a plugin.
+    structure of the editor. They **must not** introduce block
+    widgets (that will raise an error) or replacing decorations that
+    cover line breaks (these will be ignored if they occur). Such
+    decorations, or others that cause a large amount of vertical
+    size shift compared to the undecorated content, should be
+    provided through the state-level [`decorations`
+    facet](https://codemirror.net/6/docs/ref/#view.EditorView^decorations) instead.
     */
     static decorations: PluginField<DecorationSet>;
     /**
@@ -470,12 +472,13 @@ declare class ViewUpdate {
     */
     get viewportChanged(): boolean;
     /**
-    Indicates whether the line height in the editor changed in this update.
+    Indicates whether the height of an element in the editor changed
+    in this update.
     */
     get heightChanged(): boolean;
     /**
-    Returns true when the document changed or the size of the editor
-    or the lines or characters within it has changed.
+    Returns true when the document was modified or the size of the
+    editor, or elements within the editor, changed.
     */
     get geometryChanged(): boolean;
     /**
@@ -1153,6 +1156,13 @@ declare class EditorView {
         dark?: boolean;
     }): Extension;
     /**
+    This facet records whether a dark theme is active. The extension
+    returned by [`theme`](https://codemirror.net/6/docs/ref/#view.EditorView^theme) automatically
+    includes an instance of this when the `dark` option is set to
+    true.
+    */
+    static darkTheme: Facet<boolean, boolean>;
+    /**
     Create an extension that adds styles to the base theme. Like
     with [`theme`](https://codemirror.net/6/docs/ref/#view.EditorView^theme), use `&` to indicate the
     place of the editor wrapper element when directly targeting

package/dist/index.js

@@ -1307,7 +1307,7 @@ class LineView extends ContentView {
         if (attrs)
             this.attrs = combineAttrs(attrs, this.attrs || {});
         if (cls)
-            this.attrs = combineAttrs(attrs, { class: cls });
+            this.attrs = combineAttrs({ class: cls }, this.attrs || {});
     }
     domAtPos(pos) {
         return inlineDOMAtPos(this.dom, this.children, pos);
@@ -1368,16 +1368,17 @@ class LineView extends ContentView {
     become(_other) { return false; }
     get type() { return BlockType.Text; }
     static find(docView, pos) {
-        for (let i = 0, off = 0;; i++) {
+        for (let i = 0, off = 0; i < docView.children.length; i++) {
             let block = docView.children[i], end = off + block.length;
             if (end >= pos) {
                 if (block instanceof LineView)
                     return block;
-                if (block.length)
-                    return null;
+                if (end > pos)
+                    break;
             }
             off = end + block.breakAfter;
         }
+        return null;
     }
 }
 class BlockWidgetView extends ContentView {
@@ -1438,10 +1439,11 @@ class BlockWidgetView extends ContentView {
 }
 
 class ContentBuilder {
-    constructor(doc, pos, end) {
+    constructor(doc, pos, end, disallowBlockEffectsBelow) {
         this.doc = doc;
         this.pos = pos;
         this.end = end;
+        this.disallowBlockEffectsBelow = disallowBlockEffectsBelow;
         this.content = [];
         this.curLine = null;
         this.breakAtStart = 0;
@@ -1570,8 +1572,15 @@ class ContentBuilder {
         if (this.openStart < 0)
             this.openStart = openStart;
     }
-    static build(text, from, to, decorations) {
-        let builder = new ContentBuilder(text, from, to);
+    filterPoint(from, to, value, index) {
+        if (index >= this.disallowBlockEffectsBelow || !(value instanceof PointDecoration))
+            return true;
+        if (value.block)
+            throw new RangeError("Block decorations may not be specified via plugins");
+        return to <= this.doc.lineAt(this.pos).to;
+    }
+    static build(text, from, to, decorations, pluginDecorationLength) {
+        let builder = new ContentBuilder(text, from, to, pluginDecorationLength);
         builder.openEnd = RangeSet.spans(decorations, from, to, builder);
         if (builder.openStart < 0)
             builder.openStart = builder.openEnd;
@@ -1694,11 +1703,13 @@ This field can be used by plugins to provide
 **Note**: For reasons of data flow (plugins are only updated
 after the viewport is computed), decorations produced by plugins
 are _not_ taken into account when predicting the vertical layout
-structure of the editor. Thus, things like large widgets or big
-replacements (i.e. code folding) should be provided through the
-state-level [`decorations` facet](https://codemirror.net/6/docs/ref/#view.EditorView^decorations),
-not this plugin field. Specifically, replacing decorations that
-cross line boundaries will break if provided through a plugin.
+structure of the editor. They **must not** introduce block
+widgets (that will raise an error) or replacing decorations that
+cover line breaks (these will be ignored if they occur). Such
+decorations, or others that cause a large amount of vertical
+size shift compared to the undecorated content, should be
+provided through the state-level [`decorations`
+facet](https://codemirror.net/6/docs/ref/#view.EditorView^decorations) instead.
 */
 PluginField.decorations = /*@__PURE__*/PluginField.define();
 /**
@@ -1931,8 +1942,6 @@ class ViewUpdate {
             view.inputState.notifiedFocused = focus;
             this.flags |= 1 /* Focus */;
         }
-        if (this.docChanged)
-            this.flags |= 2 /* Height */;
     }
     /**
     Tells you whether the [viewport](https://codemirror.net/6/docs/ref/#view.EditorView.viewport) or
@@ -1943,14 +1952,15 @@ class ViewUpdate {
         return (this.flags & 4 /* Viewport */) > 0;
     }
     /**
-    Indicates whether the line height in the editor changed in this update.
+    Indicates whether the height of an element in the editor changed
+    in this update.
     */
     get heightChanged() {
         return (this.flags & 2 /* Height */) > 0;
     }
     /**
-    Returns true when the document changed or the size of the editor
-    or the lines or characters within it has changed.
+    Returns true when the document was modified or the size of the
+    editor, or elements within the editor, changed.
     */
     get geometryChanged() {
         return this.docChanged || (this.flags & (8 /* Geometry */ | 2 /* Height */)) > 0;
@@ -2368,6 +2378,7 @@ class DocView extends ContentView {
         this.view = view;
         this.compositionDeco = Decoration.none;
         this.decorations = [];
+        this.pluginDecorationLength = 0;
         // Track a minimum width for the editor. When measuring sizes in
         // measureVisibleLineHeights, this is updated to point at the width
         // of a given element and its extent in the document. When a change
@@ -2389,7 +2400,8 @@ class DocView extends ContentView {
         this.setDOM(view.contentDOM);
         this.children = [new LineView];
         this.children[0].setParent(this);
-        this.updateInner([new ChangedRange(0, 0, 0, view.state.doc.length)], this.updateDeco(), 0);
+        this.updateDeco();
+        this.updateInner([new ChangedRange(0, 0, 0, view.state.doc.length)], 0);
     }
     get root() { return this.view.root; }
     get editorView() { return this.view; }
@@ -2428,7 +2440,7 @@ class DocView extends ContentView {
             return false;
         }
         else {
-            this.updateInner(changedRanges, deco, update.startState.doc.length);
+            this.updateInner(changedRanges, update.startState.doc.length);
             if (update.transactions.length)
                 this.lastUpdate = Date.now();
             return true;
@@ -2436,9 +2448,9 @@ class DocView extends ContentView {
     }
     // Used by update and the constructor do perform the actual DOM
     // update
-    updateInner(changes, deco, oldLength) {
+    updateInner(changes, oldLength) {
         this.view.viewState.mustMeasureContent = true;
-        this.updateChildren(changes, deco, oldLength);
+        this.updateChildren(changes, oldLength);
         let { observer } = this.view;
         observer.ignore(() => {
             // Lock the height during redrawing, since Chrome sometimes
@@ -2465,14 +2477,14 @@ class DocView extends ContentView {
                     gaps.push(child.dom);
         observer.updateGaps(gaps);
     }
-    updateChildren(changes, deco, oldLength) {
+    updateChildren(changes, oldLength) {
         let cursor = this.childCursor(oldLength);
         for (let i = changes.length - 1;; i--) {
             let next = i >= 0 ? changes[i] : null;
             if (!next)
                 break;
             let { fromA, toA, fromB, toB } = next;
-            let { content, breakAtStart, openStart, openEnd } = ContentBuilder.build(this.view.state.doc, fromB, toB, deco);
+            let { content, breakAtStart, openStart, openEnd } = ContentBuilder.build(this.view.state.doc, fromB, toB, this.decorations, this.pluginDecorationLength);
             let { i: toI, off: toOff } = cursor.findPos(toA, 1);
             let { i: fromI, off: fromOff } = cursor.findPos(fromA, -1);
             replaceRange(this, fromI, fromOff, toI, toOff, content, breakAtStart, openStart, openEnd);
@@ -2691,8 +2703,10 @@ class DocView extends ContentView {
         return Decoration.set(deco);
     }
     updateDeco() {
+        let pluginDecorations = this.view.pluginField(PluginField.decorations);
+        this.pluginDecorationLength = pluginDecorations.length;
         return this.decorations = [
-            ...this.view.pluginField(PluginField.decorations),
+            ...pluginDecorations,
             ...this.view.state.facet(decorations),
             this.compositionDeco,
             this.computeBlockGapDeco(),
@@ -2986,7 +3000,11 @@ function posAtCoords(view, { x, y }, precise, bias = -1) {
     var _a;
     let content = view.contentDOM.getBoundingClientRect(), docTop = content.top + view.viewState.paddingTop;
     let block, { docHeight } = view.viewState;
-    let yOffset = Math.max(0, Math.min(y - docTop, docHeight));
+    let yOffset = y - docTop;
+    if (yOffset < 0)
+        return 0;
+    if (yOffset > docHeight)
+        return view.state.doc.length;
     // Scan for a text block near the queried y position
     for (let halfLine = view.defaultLineHeight / 2, bounced = false;;) {
         block = view.elementAtHeight(yOffset);
@@ -3009,10 +3027,10 @@ function posAtCoords(view, { x, y }, precise, bias = -1) {
     let lineStart = block.from;
     // If this is outside of the rendered viewport, we can't determine a position
     if (lineStart < view.viewport.from)
-        return view.viewport.from == 0 ? 0 : precise ? posAtCoordsImprecise(view, content, block, x, y) : null;
+        return view.viewport.from == 0 ? 0 : precise ? null : posAtCoordsImprecise(view, content, block, x, y);
     if (lineStart > view.viewport.to)
         return view.viewport.to == view.state.doc.length ? view.state.doc.length :
-            precise ? posAtCoordsImprecise(view, content, block, x, y) : null;
+            precise ? null : posAtCoordsImprecise(view, content, block, x, y);
     // Prefer ShadowRootOrDocument.elementFromPoint if present, fall back to document if not
     let doc = view.dom.ownerDocument;
     let root = view.root.elementFromPoint ? view.root : doc;
@@ -3047,6 +3065,8 @@ function posAtCoords(view, { x, y }, precise, bias = -1) {
     // No luck, do our own (potentially expensive) search
     if (!node || !view.docView.dom.contains(node)) {
         let line = LineView.find(view.docView, lineStart);
+        if (!line)
+            return yOffset > block.top + block.height / 2 ? block.to : block.from;
         ({ node, offset } = domPosAtCoords(line.dom, x, y));
     }
     return view.docView.posFromDOM(node, offset);
@@ -4573,6 +4593,7 @@ class ViewState {
         this.contentDOMWidth = 0;
         this.contentDOMHeight = 0;
         this.editorHeight = 0;
+        this.editorWidth = 0;
         this.heightOracle = new HeightOracle;
         // See VP.MaxDOMHeight
         this.scaler = IdScaler;
@@ -4678,11 +4699,18 @@ class ViewState {
         }
         if (!this.inView)
             return 0;
+        let contentWidth = dom.clientWidth;
+        if (this.contentDOMWidth != contentWidth || this.editorHeight != view.scrollDOM.clientHeight ||
+            this.editorWidth != view.scrollDOM.clientWidth) {
+            this.contentDOMWidth = contentWidth;
+            this.editorHeight = view.scrollDOM.clientHeight;
+            this.editorWidth = view.scrollDOM.clientWidth;
+            result |= 8 /* Geometry */;
+        }
         if (measureContent) {
             let lineHeights = view.docView.measureVisibleLineHeights();
             if (oracle.mustRefreshForHeights(lineHeights))
                 refresh = true;
-            let contentWidth = dom.clientWidth;
             if (refresh || oracle.lineWrapping && Math.abs(contentWidth - this.contentDOMWidth) > oracle.charWidth) {
                 let { lineHeight, charWidth } = view.docView.measureTextSize();
                 refresh = oracle.refresh(whiteSpace, direction, lineHeight, charWidth, contentWidth / charWidth, lineHeights);
@@ -4691,14 +4719,6 @@ class ViewState {
                     result |= 8 /* Geometry */;
                 }
             }
-            if (this.contentDOMWidth != contentWidth) {
-                this.contentDOMWidth = contentWidth;
-                result |= 8 /* Geometry */;
-            }
-            if (this.editorHeight != view.scrollDOM.clientHeight) {
-                this.editorHeight = view.scrollDOM.clientHeight;
-                result |= 8 /* Geometry */;
-            }
             if (dTop > 0 && dBottom > 0)
                 bias = Math.max(dTop, dBottom);
             else if (dTop < 0 && dBottom < 0)
@@ -5838,7 +5858,7 @@ class EditorView {
         if (state.facet(EditorState.phrases) != this.state.facet(EditorState.phrases))
             return this.setState(state);
         update = new ViewUpdate(this, state, transactions);
-        let scrollTarget = null;
+        let scrollTarget = this.viewState.scrollTarget;
         try {
             this.updateState = 2 /* Updating */;
             for (let tr of transactions) {
@@ -6559,6 +6579,13 @@ mechanism for providing decorations.
 */
 EditorView.decorations = decorations;
 /**
+This facet records whether a dark theme is active. The extension
+returned by [`theme`](https://codemirror.net/6/docs/ref/#view.EditorView^theme) automatically
+includes an instance of this when the `dark` option is set to
+true.
+*/
+EditorView.darkTheme = darkTheme;
+/**
 Facet that provides additional DOM attributes for the editor's
 editable DOM element.
 */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codemirror/view",
-  "version": "0.19.35",
+  "version": "0.19.39",
   "description": "DOM view component for the CodeMirror code editor",
   "scripts": {
     "test": "cm-runtests",
@@ -26,7 +26,7 @@
   "sideEffects": false,
   "license": "MIT",
   "dependencies": {
-    "@codemirror/rangeset": "^0.19.4",
+    "@codemirror/rangeset": "^0.19.5",
     "@codemirror/state": "^0.19.3",
     "@codemirror/text": "^0.19.0",
     "style-mod": "^4.0.0",