@portabletext/editor 1.0.18 → 1.0.19

package/lib/index.esm.js

@@ -122,7 +122,7 @@ function normalizeSelection(selection, value) {
   const { anchor, focus } = selection;
   return anchor && value.find((blk) => isEqual({ _key: blk._key }, anchor.path[0])) && (newAnchor = normalizePoint(anchor, value)), focus && value.find((blk) => isEqual({ _key: blk._key }, focus.path[0])) && (newFocus = normalizePoint(focus, value)), newAnchor && newFocus ? { anchor: newAnchor, focus: newFocus, backward: selection.backward } : null;
 }
-const EMPTY_MARKDEFS = [], EMPTY_MARKS$1 = [], VOID_CHILD_KEY = "void-child";
+const EMPTY_MARKDEFS = [], VOID_CHILD_KEY = "void-child";
 function keepObjectEquality(object, keyMap) {
   const value = keyMap[object._key];
   return value && isEqual(object, value) ? value : (keyMap[object._key] = object, object);
@@ -1168,9 +1168,10 @@ function createWithEditableAPI(portableTextEditor, types, keyGenerator) {
           const [block] = Editor.node(editor, originalSelection.focus, { depth: 1 });
           if (!editor.isTextBlock(block))
             return;
-          Range.isCollapsed(originalSelection) && (editor.pteExpandToWord(), editor.onChange());
           const [textNode] = Editor.node(editor, originalSelection.focus, { depth: 2 });
-          editor.selection && (Editor.withoutNormalizing(editor, () => {
+          if (!isPortableTextSpan$1(textNode) || textNode.text === "")
+            return;
+          Range.isCollapsed(originalSelection) && (editor.pteExpandToWord(), editor.onChange()), editor.selection && (Editor.withoutNormalizing(editor, () => {
             const annotationKey = keyGenerator();
             Transforms.setNodes(
               editor,
@@ -2786,7 +2787,7 @@ function createWithPortableTextMarkModel(types, change$, keyGenerator) {
       change$.next({ type: "selection", selection: ptRange });
     };
     return editor.normalizeNode = (nodeEntry) => {
-      const [node, path] = nodeEntry, isSpan = Text.isText(node) && node._type === types.span.name, isTextBlock = editor.isTextBlock(node);
+      const [node, path] = nodeEntry;
       if (editor.isTextBlock(node)) {
         const children = Node.children(editor, path);
         for (const [child, childPath] of children) {
@@ -2801,65 +2802,47 @@ function createWithPortableTextMarkModel(types, change$, keyGenerator) {
           }
         }
       }
-      if (isSpan || isTextBlock) {
-        if (isSpan && !Array.isArray(node.marks)) {
-          debug$c("Adding .marks to span node"), Transforms.setNodes(editor, { marks: [] }, { at: path });
-          return;
-        }
-        if (isSpan && (node.marks || []).length > 0) {
-          const spanMarks = node.marks || EMPTY_MARKS$1, annotationMarks = spanMarks.filter(
-            (mark) => !types.decorators.map((dec) => dec.value).includes(mark)
+      if (editor.isTextSpan(node) && !Array.isArray(node.marks)) {
+        debug$c("Adding .marks to span node"), Transforms.setNodes(editor, { marks: [] }, { at: path });
+        return;
+      }
+      if (editor.isTextSpan(node)) {
+        const blockPath = Path.parent(path), [block] = Editor.node(editor, blockPath), decorators2 = types.decorators.map((decorator) => decorator.value), annotations = node.marks?.filter((mark) => !decorators2.includes(mark));
+        if (editor.isTextBlock(block) && node.text === "" && annotations && annotations.length > 0) {
+          debug$c("Removing annotations from empty span node"), Transforms.setNodes(
+            editor,
+            { marks: node.marks?.filter((mark) => decorators2.includes(mark)) },
+            { at: path }
           );
-          if (annotationMarks.length > 0) {
-            const [block] = Editor.node(editor, Path.parent(path)), orphanedMarks = editor.isTextBlock(block) && annotationMarks.filter(
-              (mark) => !block.markDefs?.find((def) => def._key === mark)
-            ) || [];
-            if (orphanedMarks.length > 0) {
-              debug$c("Removing orphaned .marks from span node"), Transforms.setNodes(
-                editor,
-                { marks: spanMarks.filter((mark) => !orphanedMarks.includes(mark)) },
-                { at: path }
-              );
-              return;
-            }
-          }
+          return;
         }
-        for (const op of editor.operations) {
-          if (op.type === "merge_node" && op.path.length === 1 && "markDefs" in op.properties && op.properties._type === types.block.name && Array.isArray(op.properties.markDefs) && op.properties.markDefs.length > 0 && op.path[0] - 1 >= 0) {
-            const [targetBlock, targetPath] = Editor.node(editor, [op.path[0] - 1]);
-            if (debug$c("Copying markDefs over to merged block", op), editor.isTextBlock(targetBlock)) {
-              const oldDefs = Array.isArray(targetBlock.markDefs) && targetBlock.markDefs || [], newMarkDefs = uniq([...oldDefs, ...op.properties.markDefs]);
-              if (!isEqual(newMarkDefs, targetBlock.markDefs)) {
-                Transforms.setNodes(editor, { markDefs: newMarkDefs }, { at: targetPath, voids: !1 });
-                return;
-              }
-            }
-          }
-          if (op.type === "split_node" && op.path.length === 1 && Element$1.isElementProps(op.properties) && op.properties._type === types.block.name && "markDefs" in op.properties && Array.isArray(op.properties.markDefs) && op.properties.markDefs.length > 0 && op.path[0] + 1 < editor.children.length) {
-            const [targetBlock, targetPath] = Editor.node(editor, [op.path[0] + 1]);
-            if (debug$c("Copying markDefs over to split block", op), editor.isTextBlock(targetBlock)) {
-              const oldDefs = Array.isArray(targetBlock.markDefs) && targetBlock.markDefs || [];
-              Transforms.setNodes(
+      }
+      if (editor.isTextBlock(node)) {
+        const decorators2 = types.decorators.map((decorator) => decorator.value);
+        for (const [child, childPath] of Node.children(editor, path))
+          if (editor.isTextSpan(child)) {
+            const marks = child.marks ?? [], orphanedAnnotations = marks.filter((mark) => !decorators2.includes(mark) && !node.markDefs?.find((def) => def._key === mark));
+            if (orphanedAnnotations.length > 0) {
+              debug$c("Removing orphaned annotations from span node"), Transforms.setNodes(
                 editor,
-                { markDefs: uniq([...oldDefs, ...op.properties.markDefs]) },
-                { at: targetPath, voids: !1 }
+                { marks: marks.filter((mark) => !orphanedAnnotations.includes(mark)) },
+                { at: childPath }
               );
               return;
             }
           }
-          if (op.type === "split_node" && op.path.length === 2 && op.properties._type === types.span.name && "marks" in op.properties && Array.isArray(op.properties.marks) && op.properties.marks.length > 0 && op.path[0] + 1 < editor.children.length) {
-            const [child, childPath] = Editor.node(editor, [op.path[0] + 1, 0]);
-            if (Text.isText(child) && child.text === "" && Array.isArray(child.marks) && child.marks.length > 0) {
-              Transforms.setNodes(editor, { marks: [] }, { at: childPath, voids: !1 });
-              return;
-            }
-          }
-          if (op.type === "split_node" && op.path.length === 1 && op.properties._type === types.block.name && "markDefs" in op.properties && Array.isArray(op.properties.markDefs) && op.properties.markDefs.length > 0) {
-            const [block, blockPath] = Editor.node(editor, [op.path[0]]);
-            if (editor.isTextBlock(block) && block.children.length === 1 && block.markDefs && block.markDefs.length > 0 && Text.isText(block.children[0]) && block.children[0].text === "" && (!block.children[0].marks || block.children[0].marks.length === 0)) {
-              Transforms.setNodes(editor, { markDefs: [] }, { at: blockPath });
-              return;
-            }
+      }
+      if (editor.isTextSpan(node)) {
+        const blockPath = Path.parent(path), [block] = Editor.node(editor, blockPath);
+        if (editor.isTextBlock(block)) {
+          const decorators2 = types.decorators.map((decorator) => decorator.value), marks = node.marks ?? [], orphanedAnnotations = marks.filter((mark) => !decorators2.includes(mark) && !block.markDefs?.find((def) => def._key === mark));
+          if (orphanedAnnotations.length > 0) {
+            debug$c("Removing orphaned annotations from span node"), Transforms.setNodes(
+              editor,
+              { marks: marks.filter((mark) => !orphanedAnnotations.includes(mark)) },
+              { at: path }
+            );
+            return;
           }
         }
       }
@@ -2953,6 +2936,14 @@ function createWithPortableTextMarkModel(types, change$, keyGenerator) {
           }
         }
       }
+      if (op.type === "merge_node" && op.path.length === 1 && "markDefs" in op.properties && op.properties._type === types.block.name && Array.isArray(op.properties.markDefs) && op.properties.markDefs.length > 0 && op.path[0] - 1 >= 0) {
+        const [targetBlock, targetPath] = Editor.node(editor, [op.path[0] - 1]);
+        if (editor.isTextBlock(targetBlock)) {
+          const oldDefs = Array.isArray(targetBlock.markDefs) && targetBlock.markDefs || [], newMarkDefs = uniq([...oldDefs, ...op.properties.markDefs]);
+          debug$c("Copying markDefs over to merged block", op), Transforms.setNodes(editor, { markDefs: newMarkDefs }, { at: targetPath, voids: !1 }), apply2(op);
+          return;
+        }
+      }
       apply2(op);
     }, editor.addMark = (mark) => {
       if (editor.selection) {