Merge pull request udecode#382 from dylans/options-removerootlistitem

verify list options passed via removeRootListItem codepath
devpaul · Dec 9, 2020 · 064224d · 064224d
2 parents 85d9acd + ff3a719
commit 064224d
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 10 deletions.
diff --git a/...e-plugins/src/elements/list/transforms/moveListItemSublistItemsToListItemSublist.spec.tsx b/...e-plugins/src/elements/list/transforms/moveListItemSublistItemsToListItemSublist.spec.tsx
@@ -69,10 +69,14 @@ describe('when there is toListItem sublist', () => {
     const toListItem = getNodeById(editor, '11') as NodeEntry<Ancestor>;
 
     if (fromListItem && toListItem) {
-      moveListItemSublistItemsToListItemSublist(editor, {
-        fromListItem,
-        toListItem,
-      });
+      moveListItemSublistItemsToListItemSublist(
+        editor,
+        {
+          fromListItem,
+          toListItem,
+        },
+        {}
+      );
     }
 
     expect(input.children).toEqual(output.children);
@@ -129,10 +133,14 @@ describe('when there is no list in toListItem', () => {
     const toListItem = getNodeById(editor, '11') as NodeEntry<Ancestor>;
 
     if (fromListItem && toListItem) {
-      moveListItemSublistItemsToListItemSublist(editor, {
-        fromListItem,
-        toListItem,
-      });
+      moveListItemSublistItemsToListItemSublist(
+        editor,
+        {
+          fromListItem,
+          toListItem,
+        },
+        {}
+      );
     }
 
     expect(input.children).toEqual(output.children);

diff --git a/packages/slate-plugins/src/elements/list/transforms/removeRootListItem.ts b/packages/slate-plugins/src/elements/list/transforms/removeRootListItem.ts
@@ -43,7 +43,7 @@ export const removeRootListItem = (
     moveListItemSublistItemsToListItemSublist(editor, {
       fromListItem: listItem,
       toListItem: [previousListItemNode as Ancestor, previousListItemPath],
-    });
+    }, options);
 
     // Select the P tag at the previous list item
     Transforms.select(
@@ -57,7 +57,7 @@ export const removeRootListItem = (
       fromListItem: listItem,
       toList: list,
       // start: true,
-    });
+    }, options);
   }
 
   // Remove the list-item