prosemirror-transform 1.3.4 → 1.4.2

package/dist/index.js

@@ -146,6 +146,8 @@ StepMap.offset = function offset (n) {
   return n == 0 ? StepMap.empty : new StepMap(n < 0 ? [0, -n, 0] : [0, 0, n])
 };
 
+// :: StepMap
+// A StepMap that contains no changed ranges.
 StepMap.empty = new StepMap([]);
 
 // :: class extends Mappable
@@ -1439,7 +1441,7 @@ Fitter.prototype.placeNodes = function placeNodes (ref) {
 };
 
 Fitter.prototype.mustMoveInline = function mustMoveInline () {
-  if (!this.$to.parent.isTextblock || this.$to.end() == this.$to.pos) { return -1 }
+  if (!this.$to.parent.isTextblock) { return -1 }
   var top = this.frontier[this.depth], level;
   if (!top.type.isTextblock || !contentAfterFits(this.$to, this.$to.depth, top.type, top.match, false) ||
       (this.$to.depth == this.depth && (level = this.findCloseLevel(this.$to)) && level.depth == this.depth)) { return -1 }
@@ -1540,6 +1542,10 @@ function invalidMarks(type, fragment, start) {
   return false
 }
 
+function definesContent(type) {
+  return type.spec.defining || type.spec.definingForContent
+}
+
 // :: (number, number, Slice) → this
 // Replace a range of the document with a given slice, using `from`,
 // `to`, and the slice's [`openStart`](#model.Slice.openStart) property
@@ -1547,9 +1553,9 @@ function invalidMarks(type, fragment, start) {
 // grow the replaced area or close open nodes in the slice in order to
 // get a fit that is more in line with WYSIWYG expectations, by
 // dropping fully covered parent nodes of the replaced region when
-// they are marked [non-defining](#model.NodeSpec.defining), or
+// they are marked [non-defining as context](#model.NodeSpec.definingAsContext), or
 // including an open parent node from the slice that _is_ marked as
-// [defining](#model.NodeSpec.defining).
+// [defining its content](#model.NodeSpec.definingForContent).
 //
 // This is the method, for example, to handle paste. The similar
 // [`replace`](#transform.Transform.replace) method is a more
@@ -1576,7 +1582,7 @@ Transform.prototype.replaceRange = function(from, to, slice) {
   // cross a defining node.
   for (var d = $from.depth, pos = $from.pos - 1; d > 0; d--, pos--) {
     var spec = $from.node(d).type.spec;
-    if (spec.defining || spec.isolating) { break }
+    if (spec.defining || spec.definingAsContext || spec.isolating) { break }
     if (targetDepths.indexOf(d) > -1) { preferredTarget = d; }
     else if ($from.before(d) == pos) { targetDepths.splice(1, 0, -d); }
   }
@@ -1591,14 +1597,14 @@ Transform.prototype.replaceRange = function(from, to, slice) {
     if (i == slice.openStart) { break }
     content = node.content;
   }
-  // Back up if the node directly above openStart, or the node above
-  // that separated only by a non-defining textblock node, is defining.
-  if (preferredDepth > 0 && leftNodes[preferredDepth - 1].type.spec.defining &&
-      $from.node(preferredTargetIndex).type != leftNodes[preferredDepth - 1].type)
-    { preferredDepth -= 1; }
-  else if (preferredDepth >= 2 && leftNodes[preferredDepth - 1].isTextblock && leftNodes[preferredDepth - 2].type.spec.defining &&
-           $from.node(preferredTargetIndex).type != leftNodes[preferredDepth - 2].type)
-    { preferredDepth -= 2; }
+
+  // Back up preferredDepth to cover defining textblocks directly
+  // above it, possibly skipping a non-defining textblock.
+  for (var d$1 = preferredDepth - 1; d$1 >= 0; d$1--) {
+    var type = leftNodes[d$1].type, def = definesContent(type);
+    if (def && $from.node(preferredTargetIndex).type != type) { preferredDepth = d$1; }
+    else if (def || !type.isTextblock) { break }
+  }
 
   for (var j = slice.openStart; j >= 0; j--) {
     var openDepth = (j + preferredDepth + 1) % (slice.openStart + 1);