collapse file click looking good 👼

web/src/components/ui/dropdown/Dropdown.tsx

@@ -85,7 +85,7 @@ export const DropdownBase = <T,>({
   closeOnSelect = true,
   onSelect,
   children,
-  align = 'center',
+  align = 'start',
   side = 'bottom',
   open,
   onOpenChange,

web/src/controllers/MetricController/MetricViewResults/MetricViewResultsController.tsx

@@ -2,7 +2,7 @@ import React, { useEffect, useMemo } from 'react';
 import type { MetricViewProps } from '../config';
 import { useMemoizedFn, useUnmount } from '@/hooks';
 import { IDataResult } from '@/api/asset_interfaces';
-import { useMetricLayout } from '../useMetricLayout';
+import { useMetricResultsLayout } from './useMetricResultsLayout';
 import { useChatLayoutContextSelector } from '@/layouts/ChatLayout/ChatLayoutContext';
 import { AppSplitterRef } from '@/components/ui/layouts';
 import { AppVerticalCodeSplitter } from '@/components/features/layouts/AppVerticalCodeSplitter';
@@ -67,11 +67,10 @@ export const MetricViewResults: React.FC<MetricViewProps> = React.memo(({ metric
     });
   });
 
-  const { defaultLayout, renderSecondary } = useMetricLayout({
+  const { defaultLayout, renderSecondary } = useMetricResultsLayout({
     selectedFileViewSecondary,
     appSplitterRef,
-    autoSaveId,
-    type: 'sql'
+    autoSaveId
   });
 
   useEffect(() => {

web/src/controllers/MetricController/MetricViewResults/useMetricResultsLayout.ts

@@ -4,29 +4,23 @@ import { type AppSplitterRef } from '@/components/ui/layouts';
 import Cookies from 'js-cookie';
 import { createAutoSaveId } from '@/components/ui/layouts/AppSplitter/helper';
 
-const defaultChartOpenLayout = ['auto', '310px'];
 const defaultSqlOpenLayout = ['30%', 'auto'];
-
-const defaultChartLayout = ['auto', '0px'];
 const defaultSqlLayout = ['0px', 'auto'];
 
-export const useMetricLayout = ({
+export const useMetricResultsLayout = ({
   selectedFileViewSecondary,
   appSplitterRef,
-  autoSaveId,
-  type
+  autoSaveId
 }: {
   selectedFileViewSecondary: null | string;
   appSplitterRef: React.RefObject<AppSplitterRef | null>;
   autoSaveId: string;
-  type: 'chart' | 'sql';
 }) => {
   const [renderSecondary, setRenderSecondary] = useState<boolean>(!!selectedFileViewSecondary);
 
   const isOpenSecondary = !!selectedFileViewSecondary;
-  const isChart = type === 'chart';
-  const defaultOpenLayout = isChart ? defaultChartOpenLayout : defaultSqlOpenLayout;
-  const defaultOriginalLayout = isChart ? defaultChartLayout : defaultSqlLayout;
+  const defaultOpenLayout = defaultSqlOpenLayout;
+  const defaultOriginalLayout = defaultSqlLayout;
 
   const secondaryLayoutDimensions: [string, string] = useMemo(() => {
     const cookieKey = createAutoSaveId(autoSaveId);
@@ -54,20 +48,10 @@ export const useMetricLayout = ({
   const animateOpenSplitter = useMemoizedFn((side: 'metric' | 'both') => {
     if (!appSplitterRef.current) return;
 
-    if (type === 'chart') {
-      if (side === 'metric') {
-        appSplitterRef.current.animateWidth('100%', 'left');
-      } else if (side === 'both') {
-        appSplitterRef.current.animateWidth('310px', 'right');
-      }
-    }
-
-    if (type === 'sql') {
-      if (side === 'metric') {
-        appSplitterRef.current.animateWidth('0px', 'left');
-      } else if (side === 'both') {
-        appSplitterRef.current.animateWidth('40%', 'left');
-      }
+    if (side === 'metric') {
+      appSplitterRef.current.animateWidth('0px', 'left');
+    } else if (side === 'both') {
+      appSplitterRef.current.animateWidth('40%', 'left');
     }
   });
 

web/src/layouts/ChatLayout/ChatLayout/ChatLayout.tsx

@@ -26,8 +26,6 @@ export const ChatLayout: React.FC<ChatSplitterProps> = ({ children }) => {
     return ['380px', 'auto'];
   }, [selectedLayout]);
 
-  console.log(selectedLayout);
-
   return (
     <ChatLayoutContextProvider chatLayoutProps={chatLayoutProps}>
       <ChatContextProvider

web/src/layouts/ChatLayout/ChatLayoutContext/ChatLayoutContext.tsx

@@ -37,12 +37,11 @@ export const useChatLayoutContext = ({ appSplitterRef }: UseLayoutConfigProps) =
     }
   });
 
-  const { onCollapseFileClick, selectedFile, onSetSelectedFile, isVersionHistoryMode } =
-    useSelectedFile({
-      animateOpenSplitter,
-      chatParams,
-      appSplitterRef
-    });
+  const { selectedFile, onSetSelectedFile, isVersionHistoryMode } = useSelectedFile({
+    animateOpenSplitter,
+    chatParams,
+    appSplitterRef
+  });
 
   const {
     selectedFileView,
@@ -50,11 +49,14 @@ export const useChatLayoutContext = ({ appSplitterRef }: UseLayoutConfigProps) =
     selectedFileViewSecondary,
     onSetFileView,
     closeSecondaryView,
-    selectedLayout
+    selectedLayout,
+    onCollapseFileClick
   } = useLayoutConfig({
     selectedFile,
     isVersionHistoryMode,
-    chatId: chatParams.chatId
+    chatId: chatParams.chatId,
+    animateOpenSplitter,
+    onSetSelectedFile
   });
 
   return {

web/src/layouts/ChatLayout/ChatLayoutContext/useLayoutConfig/useLayoutConfig.ts

@@ -7,15 +7,20 @@ import { useMemoizedFn } from '@/hooks';
 import { create } from 'mutative';
 import { ChatLayoutView } from '../../interfaces';
 import type { SelectedFile } from '../../interfaces';
+import { timeout } from '@/lib';
 
 export const useLayoutConfig = ({
   selectedFile,
   isVersionHistoryMode,
-  chatId
+  chatId,
+  onSetSelectedFile,
+  animateOpenSplitter
 }: {
   selectedFile: SelectedFile | null;
   isVersionHistoryMode: boolean;
   chatId: string | undefined;
+  animateOpenSplitter: (side: 'left' | 'right' | 'both') => void;
+  onSetSelectedFile: (file: SelectedFile | null) => void;
 }) => {
   const [fileViews, setFileViews] = useState<Record<string, FileConfig>>({});
 
@@ -48,7 +53,7 @@ export const useLayoutConfig = ({
   }, [selectedFileViewConfig]);
 
   const onSetFileView = useMemoizedFn(
-    ({
+    async ({
       fileView,
       fileId: fileIdProp,
       secondaryView,
@@ -61,6 +66,14 @@ export const useLayoutConfig = ({
     }) => {
       const fileId = fileIdProp ?? selectedFileId;
       if (!fileId) return;
+
+      if (secondaryView) {
+        animateOpenSplitter('right');
+        await timeout(250); //wait for splitter to close before opening secondary view
+      } else {
+        animateOpenSplitter('both');
+      }
+
       setFileViews((prev) => {
         return create(prev, (draft) => {
           if (!draft[fileId]) {
@@ -105,6 +118,11 @@ export const useLayoutConfig = ({
     });
   });
 
+  const onCollapseFileClick = useMemoizedFn(() => {
+    onSetSelectedFile(null);
+    closeSecondaryView();
+  });
+
   const selectedLayout: ChatLayoutView = useMemo(() => {
     if (chatId) {
       if (selectedFileId) return 'both';
@@ -128,14 +146,26 @@ export const useLayoutConfig = ({
     }
   }, [isVersionHistoryMode, selectedFileId]);
 
-  return {
-    selectedLayout,
-    selectedFileView,
-    selectedFileViewSecondary,
-    selectedFileViewRenderSecondary,
-    onSetFileView,
-    closeSecondaryView
-  };
+  return useMemo(
+    () => ({
+      selectedLayout,
+      selectedFileView,
+      selectedFileViewSecondary,
+      selectedFileViewRenderSecondary,
+      onSetFileView,
+      closeSecondaryView,
+      onCollapseFileClick
+    }),
+    [
+      selectedLayout,
+      selectedFileView,
+      selectedFileViewSecondary,
+      selectedFileViewRenderSecondary,
+      onSetFileView,
+      closeSecondaryView,
+      onCollapseFileClick
+    ]
+  );
 };
 
 const DEFAULT_FILE_VIEW: Record<FileType, FileView> = {

web/src/layouts/ChatLayout/ChatLayoutContext/useSelectedFile/useSelectedFile.ts

@@ -44,16 +44,11 @@ export const useSelectedFile = ({
     animateOpenSplitter('both');
   });
 
-  const onCollapseFileClick = useMemoizedFn((close?: boolean) => {
-    onSetSelectedFile(null);
-  });
-
   return useMemo(
     () => ({
       isVersionHistoryMode,
       onSetSelectedFile,
-      selectedFile,
-      onCollapseFileClick
+      selectedFile
     }),
     [onSetSelectedFile, isVersionHistoryMode, selectedFile]
   );

web/src/layouts/ChatLayout/FileContainer/FileContainer.tsx

@@ -78,7 +78,7 @@ export const FileContainer: React.FC<FileContainerProps> = ({ children }) => {
   useLayoutEffect(() => {
     setTimeout(() => {
       animateOpenSplitter(isOpenSecondary ? 'open' : 'closed');
-    }, 25);
+    }, 20);
   }, [isOpenSecondary]);
 
   return (

web/src/layouts/ChatLayout/FileContainer/FileContainerHeader/CollapseFileButton.tsx

@@ -2,18 +2,14 @@ import { DoubleChevronRight } from '@/components/ui/icons';
 import { Button } from '@/components/ui/buttons';
 import React, { useMemo } from 'react';
 import { AnimatePresence, motion } from 'framer-motion';
-import { useMemoizedFn } from '@/hooks';
+import { AppTooltip } from '@/components/ui/tooltip';
 
 export const CollapseFileButton: React.FC<{
   showCollapseButton: boolean;
-  onCollapseFileClick: (value?: boolean) => void;
+  onCollapseFileClick: () => void;
 }> = React.memo(({ showCollapseButton, onCollapseFileClick }) => {
   const icon = <DoubleChevronRight />;
 
-  const onClick = useMemoizedFn(() => {
-    onCollapseFileClick();
-  });
-
   const animation = useMemo(() => {
     return {
       initial: { opacity: 0 },
@@ -26,7 +22,9 @@ export const CollapseFileButton: React.FC<{
     <AnimatePresence mode="wait" initial={false}>
       {showCollapseButton && (
         <motion.div variants={animation}>
-          <Button onClick={onClick} variant="ghost" prefix={icon}></Button>
+          <AppTooltip title="Collapse file" delayDuration={350}>
+            <Button onClick={onCollapseFileClick} variant="ghost" prefix={icon}></Button>
+          </AppTooltip>
         </motion.div>
       )}
     </AnimatePresence>

web/src/layouts/ChatLayout/FileContainer/FileContainerHeader/CreateChatButtont.tsx

@@ -7,10 +7,9 @@ import { useMemoizedFn } from '@/hooks';
 import { AppTooltip } from '@/components/ui/tooltip';
 
 export const CreateChatButton = React.memo(() => {
-  const onCollapseFileClick = useChatLayoutContextSelector((x) => x.onCollapseFileClick);
-
   const onCollapseFileClickPreflight = useMemoizedFn(() => {
-    onCollapseFileClick(false);
+    //   onCollapseFileClick(false);
+    alert('TODO');
   });
 
   useHotkeys('e', onCollapseFileClickPreflight, { preventDefault: true });