[WEB-5306] refactor: base layouts (#8051)

Author: JayashTripathy

apps/web/core/components/base-layouts/kanban/layout.tsx

@@ -20,15 +20,22 @@ export const BaseKanbanLayout = observer(<T extends IBaseLayoutsKanbanItem>(prop
     showEmptyGroups = true,
     enableDragDrop = false,
     loadMoreItems,
-    collapsedGroups: externalCollapsedGroups = [],
-    onToggleGroup: externalOnToggleGroup = () => {},
+    collapsedGroups: externalCollapsedGroups,
+    onToggleGroup: externalOnToggleGroup,
   } = props;
 
-  const { containerRef, collapsedGroups, onToggleGroup } = useLayoutState({
-    mode: "external",
-    externalCollapsedGroups,
-    externalOnToggleGroup,
-  });
+  const useExternalMode = externalCollapsedGroups !== undefined && externalOnToggleGroup !== undefined;
+  const { containerRef, collapsedGroups, onToggleGroup } = useLayoutState(
+    useExternalMode
+      ? {
+          mode: "external",
+          externalCollapsedGroups,
+          externalOnToggleGroup,
+        }
+      : {
+          mode: "internal",
+        }
+  );
 
   return (
     <div ref={containerRef} className={cn("relative w-full flex gap-2 p-3 h-full overflow-x-auto", className)}>

apps/web/core/components/base-layouts/layout-switcher.tsx

@@ -9,7 +9,7 @@ import { BASE_LAYOUTS } from "./constants";
 type Props = {
   layouts?: TBaseLayoutType[];
   onChange: (layout: TBaseLayoutType) => void;
-  selectedLayout: TBaseLayoutType | undefined;
+  selectedLayout: TBaseLayoutType;
 };
 
 export const LayoutSwitcher: React.FC<Props> = (props) => {

apps/web/core/components/base-layouts/list/layout.tsx

@@ -17,17 +17,24 @@ export const BaseListLayout = observer(<T extends IBaseLayoutsListItem>(props: I
     onDrop,
     canDrag,
     showEmptyGroups = false,
-    collapsedGroups: externalCollapsedGroups = [],
-    onToggleGroup: externalOnToggleGroup = () => {},
+    collapsedGroups: externalCollapsedGroups,
+    onToggleGroup: externalOnToggleGroup,
     loadMoreItems,
     className,
   } = props;
 
-  const { containerRef, collapsedGroups, onToggleGroup } = useLayoutState({
-    mode: "external",
-    externalCollapsedGroups,
-    externalOnToggleGroup,
-  });
+  const useExternalMode = externalCollapsedGroups !== undefined && externalOnToggleGroup !== undefined;
+  const { containerRef, collapsedGroups, onToggleGroup } = useLayoutState(
+    useExternalMode
+      ? {
+          mode: "external",
+          externalCollapsedGroups,
+          externalOnToggleGroup,
+        }
+      : {
+          mode: "internal",
+        }
+  );
 
   return (
     <div ref={containerRef} className={cn("relative size-full overflow-auto bg-custom-background-90", className)}>