chore: fix auto-flow issue (#5024)

fix auto-flow issue
cds-snc · Jan 23, 2025 · 11df4fc · 11df4fc
1 parent 4c9bcf2
commit 11df4fc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/...e]/(form administration)/form-builder/components/shared/right-panel/treeview/TreeView.tsx b/...e]/(form administration)/form-builder/components/shared/right-panel/treeview/TreeView.tsx
@@ -319,11 +319,11 @@ const ControlledTree: ForwardRefRenderFunction<unknown, TreeDataProviderProps> =
             expandedItems,
             getTreeData,
             getConfirmMovePromise,
-            setOpenConfirmMoveDialog,
-            autoFlowAll
+            setOpenConfirmMoveDialog
           );
 
           updateGroupsLayout();
+          autoFlowAll();
           forceRefresh();
         }}
         onFocusItem={(item) => {

diff --git a/...inistration)/form-builder/components/shared/right-panel/treeview/handlers/handleOnDrop.ts b/...inistration)/form-builder/components/shared/right-panel/treeview/handlers/handleOnDrop.ts
@@ -118,8 +118,7 @@ export const handleOnDrop = async (
   expandedItems: TreeItemIndex[],
   getTreeData: () => TreeItems,
   getPromise: () => Promise<boolean>,
-  setOpenDialog: (value: boolean) => void,
-  autoFlowAll: () => void
+  setOpenDialog: (value: boolean) => void
 ) => {
   // Current state of the tree in Groups format
   let currentGroups = getGroups() as GroupsType;
@@ -217,10 +216,6 @@ export const handleOnDrop = async (
 
     replaceGroups(newGroups);
     setSelectedItems(selectedItems);
-
-    // Temporary solution: If no custom rules are present, autoFlow all items
-    autoFlowAll();
-
     return;
   }