prosemirror-transform 1.7.4 → 1.7.5

package/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 1.7.5 (2023-08-22)
+
+### Bug fixes
+
+Fix a failure in `replaceRange` to drop wrapper nodes when the same wrapper is already present.
+
 ## 1.7.4 (2023-07-28)
 
 ### Bug fixes

package/dist/index.cjs

@@ -28,10 +28,6 @@ function _defineProperties(target, props) { for (var i = 0; i < props.length; i+
 
 function _createClass(Constructor, protoProps, staticProps) { if (protoProps) _defineProperties(Constructor.prototype, protoProps); if (staticProps) _defineProperties(Constructor, staticProps); Object.defineProperty(Constructor, "prototype", { writable: false }); return Constructor; }
 
-Object.defineProperty(exports, '__esModule', {
-  value: true
-});
-
 var prosemirrorModel = require('prosemirror-model');
 
 var lower16 = 0xffff;
@@ -1663,9 +1659,9 @@ function _replaceRange(tr, from, to, slice) {
   }
 
   for (var _d3 = preferredDepth - 1; _d3 >= 0; _d3--) {
-    var type = leftNodes[_d3].type,
-        def = definesContent(type);
-    if (def && $from.node(preferredTargetIndex).type != type) preferredDepth = _d3;else if (def || !type.isTextblock) break;
+    var leftNode = leftNodes[_d3],
+        def = definesContent(leftNode.type);
+    if (def && !leftNode.sameMarkup($from.node(Math.abs(preferredTarget) - 1))) preferredDepth = _d3;else if (def || !leftNode.type.isTextblock) break;
   }
 
   for (var j = slice.openStart; j >= 0; j--) {
@@ -1828,17 +1824,17 @@ var AttrStep = function (_Step7) {
 Step.jsonID("attr", AttrStep);
 
 exports.TransformError = function (_Error) {
-  _inherits(_class, _Error);
+  _inherits(TransformError, _Error);
 
-  var _super8 = _createSuper(_class);
+  var _super8 = _createSuper(TransformError);
 
-  function _class() {
-    _classCallCheck(this, _class);
+  function TransformError() {
+    _classCallCheck(this, TransformError);
 
     return _super8.apply(this, arguments);
   }
 
-  return _createClass(_class);
+  return _createClass(TransformError);
 }(_wrapNativeSuper(Error));
 
 exports.TransformError = function TransformError(message) {

package/dist/index.js

@@ -1601,10 +1601,10 @@ function replaceRange(tr, from, to, slice) {
     // Back up preferredDepth to cover defining textblocks directly
     // above it, possibly skipping a non-defining textblock.
     for (let d = preferredDepth - 1; d >= 0; d--) {
-        let type = leftNodes[d].type, def = definesContent(type);
-        if (def && $from.node(preferredTargetIndex).type != type)
+        let leftNode = leftNodes[d], def = definesContent(leftNode.type);
+        if (def && !leftNode.sameMarkup($from.node(Math.abs(preferredTarget) - 1)))
             preferredDepth = d;
-        else if (def || !type.isTextblock)
+        else if (def || !leftNode.type.isTextblock)
             break;
     }
     for (let j = slice.openStart; j >= 0; j--) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prosemirror-transform",
-  "version": "1.7.4",
+  "version": "1.7.5",
   "description": "ProseMirror document transformations",
   "type": "module",
   "main": "dist/index.cjs",

package/src/replace.ts

@@ -359,7 +359,7 @@ export function replaceRange(tr: Transform, from: number, to: number, slice: Sli
   // target depth, starting with the preferred depths.
   let preferredTargetIndex = targetDepths.indexOf(preferredTarget)
 
-  let leftNodes = [], preferredDepth = slice.openStart
+  let leftNodes: Node[] = [], preferredDepth = slice.openStart
   for (let content = slice.content, i = 0;; i++) {
     let node = content.firstChild!
     leftNodes.push(node)
@@ -370,9 +370,9 @@ export function replaceRange(tr: Transform, from: number, to: number, slice: Sli
   // Back up preferredDepth to cover defining textblocks directly
   // above it, possibly skipping a non-defining textblock.
   for (let d = preferredDepth - 1; d >= 0; d--) {
-    let type = leftNodes[d].type, def = definesContent(type)
-    if (def && $from.node(preferredTargetIndex).type != type) preferredDepth = d
-    else if (def || !type.isTextblock) break
+    let leftNode = leftNodes[d], def = definesContent(leftNode.type)
+    if (def && !leftNode.sameMarkup($from.node(Math.abs(preferredTarget) - 1))) preferredDepth = d
+    else if (def || !leftNode.type.isTextblock) break
   }
 
   for (let j = slice.openStart; j >= 0; j--) {
@@ -443,7 +443,7 @@ export function deleteRange(tr: Transform, from: number, to: number) {
 // Returns an array of all depths for which $from - $to spans the
 // whole content of the nodes at that depth.
 function coveredDepths($from: ResolvedPos, $to: ResolvedPos) {
-  let result = [], minDepth = Math.min($from.depth, $to.depth)
+  let result: number[] = [], minDepth = Math.min($from.depth, $to.depth)
   for (let d = minDepth; d >= 0; d--) {
     let start = $from.start(d)
     if (start < $from.pos - ($from.depth - d) ||