prosemirror-view 1.40.0 → 1.41.0

package/CHANGELOG.md

@@ -1,3 +1,25 @@
+## 1.41.0 (2025-09-06)
+
+### Bug fixes
+
+Make sure `transformPasted` is properly called when pasting into a code block.
+
+Fix an issue where the editor didn't properly set the DOM selection in a shadow DOM on Safari in some situations.
+
+Work around a Safari issue that broke composition right after changing the set of active marks.
+
+### New features
+
+Pass an argument to `transformPasted` that tells the handler whether the paste is plain text. Call transformPasted when pasting into a code block
+
+## 1.40.1 (2025-07-18)
+
+### Bug fixes
+
+Fix an issue where, for some Android virtual keyboards, joining textblocks that contain an image with Backspace would run the handler for Enter instead.
+
+Fix an issue causing `posAtCoords` to get confused by table cells sticking out of their parent row elements.
+
 ## 1.40.0 (2025-06-02)
 
 ### Bug fixes

package/dist/index.cjs

@@ -403,7 +403,7 @@ function posFromCaret(view, node, offset, coords) {
       rect = void 0;
     if (!desc) return null;
     if (desc.dom.nodeType == 1 && (desc.node.isBlock && desc.parent || !desc.contentDOM) && ((rect = desc.dom.getBoundingClientRect()).width || rect.height)) {
-      if (desc.node.isBlock && desc.parent) {
+      if (desc.node.isBlock && desc.parent && !/^T(R|BODY|HEAD|FOOT)$/.test(desc.dom.nodeName)) {
         if (!sawBlock && rect.left > coords.left || rect.top > coords.top) outsideBlock = desc.posBefore;else if (!sawBlock && rect.right < coords.left || rect.bottom < coords.top) outsideBlock = desc.posAfter;
         sawBlock = true;
       }
@@ -998,7 +998,7 @@ var ViewDesc = function () {
       }
       if (!(force || brKludge && safari) && isEquivalentPosition(anchorDOM.node, anchorDOM.offset, selRange.anchorNode, selRange.anchorOffset) && isEquivalentPosition(headDOM.node, headDOM.offset, selRange.focusNode, selRange.focusOffset)) return;
       var domSelExtended = false;
-      if ((domSel.extend || anchor == head) && !brKludge) {
+      if ((domSel.extend || anchor == head) && !(brKludge && gecko)) {
         domSel.collapse(anchorDOM.node, anchorDOM.offset);
         try {
           if (anchor != head) domSel.extend(headDOM.node, headDOM.offset);
@@ -2271,15 +2271,11 @@ function removeClassOnSelectionChange(view) {
   });
 }
 function selectCursorWrapper(view) {
-  var domSel = view.domSelection(),
-    range = document.createRange();
+  var domSel = view.domSelection();
   if (!domSel) return;
   var node = view.cursorWrapper.dom,
     img = node.nodeName == "IMG";
-  if (img) range.setStart(node.parentNode, domIndex(node) + 1);else range.setStart(node, 0);
-  range.collapse(true);
-  domSel.removeAllRanges();
-  domSel.addRange(range);
+  if (img) domSel.collapse(node.parentNode, domIndex(node) + 1);else domSel.collapse(node, 0);
   if (!img && !view.state.selection.visible && ie && ie_version <= 11) {
     node.disabled = true;
     node.disabled = false;
@@ -2677,12 +2673,18 @@ function parseFromClipboard(view, text, html, plainText, $context) {
   var inCode = $context.parent.type.spec.code;
   var dom, slice;
   if (!html && !text) return null;
-  var asText = text && (plainText || inCode || !html);
+  var asText = !!text && (plainText || inCode || !html);
   if (asText) {
     view.someProp("transformPastedText", function (f) {
       text = f(text, inCode || plainText, view);
     });
-    if (inCode) return text ? new prosemirrorModel.Slice(prosemirrorModel.Fragment.from(view.state.schema.text(text.replace(/\r\n?/g, "\n"))), 0, 0) : prosemirrorModel.Slice.empty;
+    if (inCode) {
+      slice = new prosemirrorModel.Slice(prosemirrorModel.Fragment.from(view.state.schema.text(text.replace(/\r\n?/g, "\n"))), 0, 0);
+      view.someProp("transformPasted", function (f) {
+        slice = f(slice, view, true);
+      });
+      return slice;
+    }
     var parsed = view.someProp("clipboardTextParser", function (f) {
       return f(text, $context, plainText, view);
     });
@@ -2739,7 +2741,7 @@ function parseFromClipboard(view, text, html, plainText, $context) {
     }
   }
   view.someProp("transformPasted", function (f) {
-    slice = f(slice, view);
+    slice = f(slice, view, asText);
   });
   return slice;
 }
@@ -3492,7 +3494,7 @@ editHandlers.drop = function (view, _event) {
   var slice = dragging && dragging.slice;
   if (slice) {
     view.someProp("transformPasted", function (f) {
-      slice = f(slice, view);
+      slice = f(slice, view, false);
     });
   } else {
     slice = parseFromClipboard(view, getText(event.dataTransfer), brokenClipboardAPI ? null : event.dataTransfer.getData("text/html"), false, $mouse);
@@ -4605,7 +4607,7 @@ function ruleFromNode(dom) {
   }
   return null;
 }
-var isInline = /^(a|abbr|acronym|b|bd[io]|big|br|button|cite|code|data(list)?|del|dfn|em|i|ins|kbd|label|map|mark|meter|output|q|ruby|s|samp|small|span|strong|su[bp]|time|u|tt|var)$/i;
+var isInline = /^(a|abbr|acronym|b|bd[io]|big|br|button|cite|code|data(list)?|del|dfn|em|i|img|ins|kbd|label|map|mark|meter|output|q|ruby|s|samp|small|span|strong|su[bp]|time|u|tt|var)$/i;
 function readDOMChange(view, from, to, typeOver, addedNodes) {
   var compositionID = view.input.compositionPendingChanges || (view.composing ? view.input.compositionID : 0);
   view.input.compositionPendingChanges = 0;

package/dist/index.d.cts

@@ -757,9 +757,10 @@ interface EditorProps<P = any> {
     clipboardTextParser?: (this: P, text: string, $context: ResolvedPos, plain: boolean, view: EditorView) => Slice;
     /**
     Can be used to transform pasted or dragged-and-dropped content
-    before it is applied to the document.
+    before it is applied to the document. The `plain` flag will be
+    true when the text is pasted as plain text.
     */
-    transformPasted?: (this: P, slice: Slice, view: EditorView) => Slice;
+    transformPasted?: (this: P, slice: Slice, view: EditorView, plain: boolean) => Slice;
     /**
     Can be used to transform copied or cut content before it is
     serialized to the clipboard.

package/dist/index.d.ts

@@ -757,9 +757,10 @@ interface EditorProps<P = any> {
     clipboardTextParser?: (this: P, text: string, $context: ResolvedPos, plain: boolean, view: EditorView) => Slice;
     /**
     Can be used to transform pasted or dragged-and-dropped content
-    before it is applied to the document.
+    before it is applied to the document. The `plain` flag will be
+    true when the text is pasted as plain text.
     */
-    transformPasted?: (this: P, slice: Slice, view: EditorView) => Slice;
+    transformPasted?: (this: P, slice: Slice, view: EditorView, plain: boolean) => Slice;
     /**
     Can be used to transform copied or cut content before it is
     serialized to the clipboard.

package/dist/index.js

@@ -414,7 +414,7 @@ function posFromCaret(view, node, offset, coords) {
         if (desc.dom.nodeType == 1 && (desc.node.isBlock && desc.parent || !desc.contentDOM) &&
             // Ignore elements with zero-size bounding rectangles
             ((rect = desc.dom.getBoundingClientRect()).width || rect.height)) {
-            if (desc.node.isBlock && desc.parent) {
+            if (desc.node.isBlock && desc.parent && !/^T(R|BODY|HEAD|FOOT)$/.test(desc.dom.nodeName)) {
                 // Only apply the horizontal test to the innermost block. Vertical for any parent.
                 if (!sawBlock && rect.left > coords.left || rect.top > coords.top)
                     outsideBlock = desc.posBefore;
@@ -1072,7 +1072,7 @@ class ViewDesc {
         // (one where the focus is before the anchor), but not all
         // browsers support it yet.
         let domSelExtended = false;
-        if ((domSel.extend || anchor == head) && !brKludge) {
+        if ((domSel.extend || anchor == head) && !(brKludge && gecko)) {
             domSel.collapse(anchorDOM.node, anchorDOM.offset);
             try {
                 if (anchor != head)
@@ -2333,17 +2333,14 @@ function removeClassOnSelectionChange(view) {
     });
 }
 function selectCursorWrapper(view) {
-    let domSel = view.domSelection(), range = document.createRange();
+    let domSel = view.domSelection();
     if (!domSel)
         return;
     let node = view.cursorWrapper.dom, img = node.nodeName == "IMG";
     if (img)
-        range.setStart(node.parentNode, domIndex(node) + 1);
+        domSel.collapse(node.parentNode, domIndex(node) + 1);
     else
-        range.setStart(node, 0);
-    range.collapse(true);
-    domSel.removeAllRanges();
-    domSel.addRange(range);
+        domSel.collapse(node, 0);
     // Kludge to kill 'control selection' in IE11 when selecting an
     // invisible cursor wrapper, since that would result in those weird
     // resize handles and a selection that considers the absolutely
@@ -2821,11 +2818,14 @@ function parseFromClipboard(view, text, html, plainText, $context) {
     let dom, slice;
     if (!html && !text)
         return null;
-    let asText = text && (plainText || inCode || !html);
+    let asText = !!text && (plainText || inCode || !html);
     if (asText) {
         view.someProp("transformPastedText", f => { text = f(text, inCode || plainText, view); });
-        if (inCode)
-            return text ? new Slice(Fragment.from(view.state.schema.text(text.replace(/\r\n?/g, "\n"))), 0, 0) : Slice.empty;
+        if (inCode) {
+            slice = new Slice(Fragment.from(view.state.schema.text(text.replace(/\r\n?/g, "\n"))), 0, 0);
+            view.someProp("transformPasted", f => { slice = f(slice, view, true); });
+            return slice;
+        }
         let parsed = view.someProp("clipboardTextParser", f => f(text, $context, plainText, view));
         if (parsed) {
             slice = parsed;
@@ -2883,7 +2883,7 @@ function parseFromClipboard(view, text, html, plainText, $context) {
             slice = closeSlice(slice, openStart, openEnd);
         }
     }
-    view.someProp("transformPasted", f => { slice = f(slice, view); });
+    view.someProp("transformPasted", f => { slice = f(slice, view, asText); });
     return slice;
 }
 const inlineParents = /^(a|abbr|acronym|b|cite|code|del|em|i|ins|kbd|label|output|q|ruby|s|samp|span|strong|sub|sup|time|u|tt|var)$/i;
@@ -3743,7 +3743,7 @@ editHandlers.drop = (view, _event) => {
     let $mouse = view.state.doc.resolve(eventPos.pos);
     let slice = dragging && dragging.slice;
     if (slice) {
-        view.someProp("transformPasted", f => { slice = f(slice, view); });
+        view.someProp("transformPasted", f => { slice = f(slice, view, false); });
     }
     else {
         slice = parseFromClipboard(view, getText(event.dataTransfer), brokenClipboardAPI ? null : event.dataTransfer.getData("text/html"), false, $mouse);
@@ -4929,7 +4929,7 @@ function ruleFromNode(dom) {
     }
     return null;
 }
-const isInline = /^(a|abbr|acronym|b|bd[io]|big|br|button|cite|code|data(list)?|del|dfn|em|i|ins|kbd|label|map|mark|meter|output|q|ruby|s|samp|small|span|strong|su[bp]|time|u|tt|var)$/i;
+const isInline = /^(a|abbr|acronym|b|bd[io]|big|br|button|cite|code|data(list)?|del|dfn|em|i|img|ins|kbd|label|map|mark|meter|output|q|ruby|s|samp|small|span|strong|su[bp]|time|u|tt|var)$/i;
 function readDOMChange(view, from, to, typeOver, addedNodes) {
     let compositionID = view.input.compositionPendingChanges || (view.composing ? view.input.compositionID : 0);
     view.input.compositionPendingChanges = 0;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prosemirror-view",
-  "version": "1.40.0",
+  "version": "1.41.0",
   "description": "ProseMirror's view component",
   "type": "module",
   "main": "dist/index.cjs",

package/src/clipboard.ts

@@ -44,10 +44,14 @@ export function parseFromClipboard(view: EditorView, text: string, html: string
   let inCode = $context.parent.type.spec.code
   let dom: HTMLElement | undefined, slice: Slice | undefined
   if (!html && !text) return null
-  let asText = text && (plainText || inCode || !html)
+  let asText = !!text && (plainText || inCode || !html)
   if (asText) {
     view.someProp("transformPastedText", f => { text = f(text, inCode || plainText, view) })
-    if (inCode) return text ? new Slice(Fragment.from(view.state.schema.text(text.replace(/\r\n?/g, "\n"))), 0, 0) : Slice.empty
+    if (inCode) {
+      slice = new Slice(Fragment.from(view.state.schema.text(text.replace(/\r\n?/g, "\n"))), 0, 0)
+      view.someProp("transformPasted", f => { slice = f(slice!, view, true) })
+      return slice
+    }
     let parsed = view.someProp("clipboardTextParser", f => f(text, $context, plainText, view))
     if (parsed) {
       slice = parsed
@@ -101,7 +105,7 @@ export function parseFromClipboard(view: EditorView, text: string, html: string
     }
   }
 
-  view.someProp("transformPasted", f => { slice = f(slice!, view) })
+  view.someProp("transformPasted", f => { slice = f(slice!, view, asText) })
   return slice
 }
 

package/src/domchange.ts

@@ -76,7 +76,7 @@ function ruleFromNode(dom: DOMNode): Omit<TagParseRule, "tag"> | null {
   return null
 }
 
-const isInline = /^(a|abbr|acronym|b|bd[io]|big|br|button|cite|code|data(list)?|del|dfn|em|i|ins|kbd|label|map|mark|meter|output|q|ruby|s|samp|small|span|strong|su[bp]|time|u|tt|var)$/i
+const isInline = /^(a|abbr|acronym|b|bd[io]|big|br|button|cite|code|data(list)?|del|dfn|em|i|img|ins|kbd|label|map|mark|meter|output|q|ruby|s|samp|small|span|strong|su[bp]|time|u|tt|var)$/i
 
 export function readDOMChange(view: EditorView, from: number, to: number, typeOver: boolean, addedNodes: readonly DOMNode[]) {
   let compositionID = view.input.compositionPendingChanges || (view.composing ? view.input.compositionID : 0)

package/src/domcoords.ts

@@ -232,7 +232,7 @@ function posFromCaret(view: EditorView, node: Node, offset: number, coords: {top
     if (desc.dom.nodeType == 1 && (desc.node.isBlock && desc.parent || !desc.contentDOM) &&
         // Ignore elements with zero-size bounding rectangles
         ((rect = (desc.dom as HTMLElement).getBoundingClientRect()).width || rect.height)) {
-      if (desc.node.isBlock && desc.parent) {
+      if (desc.node.isBlock && desc.parent && !/^T(R|BODY|HEAD|FOOT)$/.test(desc.dom!.nodeName)) {
         // Only apply the horizontal test to the innermost block. Vertical for any parent.
         if (!sawBlock && rect.left > coords.left || rect.top > coords.top) outsideBlock = desc.posBefore
         else if (!sawBlock && rect.right < coords.left || rect.bottom < coords.top) outsideBlock = desc.posAfter

package/src/index.ts

@@ -714,8 +714,9 @@ export interface EditorProps<P = any> {
   clipboardTextParser?: (this: P, text: string, $context: ResolvedPos, plain: boolean, view: EditorView) => Slice
 
   /// Can be used to transform pasted or dragged-and-dropped content
-  /// before it is applied to the document.
-  transformPasted?: (this: P, slice: Slice, view: EditorView) => Slice
+  /// before it is applied to the document. The `plain` flag will be
+  /// true when the text is pasted as plain text.
+  transformPasted?: (this: P, slice: Slice, view: EditorView, plain: boolean) => Slice
 
   /// Can be used to transform copied or cut content before it is
   /// serialized to the clipboard.

package/src/input.ts

@@ -718,7 +718,7 @@ editHandlers.drop = (view, _event) => {
   let $mouse = view.state.doc.resolve(eventPos.pos)
   let slice = dragging && dragging.slice
   if (slice) {
-    view.someProp("transformPasted", f => { slice = f(slice!, view) })
+    view.someProp("transformPasted", f => { slice = f(slice!, view, false) })
   } else {
     slice = parseFromClipboard(view, getText(event.dataTransfer),
                                brokenClipboardAPI ? null : event.dataTransfer.getData("text/html"), false, $mouse)

package/src/selection.ts

@@ -147,14 +147,11 @@ function removeClassOnSelectionChange(view: EditorView) {
 }
 
 function selectCursorWrapper(view: EditorView) {
-  let domSel = view.domSelection(), range = document.createRange()
+  let domSel = view.domSelection()
   if (!domSel) return
   let node = view.cursorWrapper!.dom, img = node.nodeName == "IMG"
-  if (img) range.setStart(node.parentNode!, domIndex(node) + 1)
-  else range.setStart(node, 0)
-  range.collapse(true)
-  domSel.removeAllRanges()
-  domSel.addRange(range)
+  if (img) domSel.collapse(node.parentNode!, domIndex(node) + 1)
+  else domSel.collapse(node, 0)
   // Kludge to kill 'control selection' in IE11 when selecting an
   // invisible cursor wrapper, since that would result in those weird
   // resize handles and a selection that considers the absolutely

package/src/viewdesc.ts

@@ -460,7 +460,7 @@ export class ViewDesc {
     // (one where the focus is before the anchor), but not all
     // browsers support it yet.
     let domSelExtended = false
-    if ((domSel.extend || anchor == head) && !brKludge) {
+    if ((domSel.extend || anchor == head) && !(brKludge && browser.gecko)) {
       domSel.collapse(anchorDOM.node, anchorDOM.offset)
       try {
         if (anchor != head)