feat: remove sidebar animate for smooth (#654)

teableio · Jun 7, 2024 · 495ca0f · 495ca0f
1 parent 402b9a6
commit 495ca0f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/apps/nextjs-app/src/features/app/components/sidebar/Sidebar.tsx b/apps/nextjs-app/src/features/app/components/sidebar/Sidebar.tsx
@@ -26,19 +26,17 @@ export const Sidebar: FC<PropsWithChildren<ISidebarProps>> = (props) => {
     <>
       {isMobile ? (
         <SheetWrapper>
-          <div className="flex size-full flex-col overflow-hidden bg-popover">
+          <div className="flex size-full flex-col overflow-hidden bg-popover p-5">
             <SidebarHeader headerLeft={headerLeft} />
             {children}
           </div>
         </SheetWrapper>
       ) : (
         <div
-          className={cn('transition-all flex w-0 border-r will-change-auto flex-shrink-0 h-full', {
+          className={cn('flex w-0 border-r flex-shrink-0 h-full', {
             'overflow-hidden': !leftVisible,
+            'w-72': leftVisible,
           })}
-          style={{
-            width: leftVisible ? `${SIDE_BAR_WIDTH}px` : '',
-          }}
         >
           <div className="flex size-full flex-col overflow-hidden bg-popover">
             <SidebarHeader headerLeft={headerLeft} onExpand={() => setLeftVisible(!leftVisible)} />

diff --git a/apps/nextjs-app/src/features/app/components/sidebar/SidebarHeader.tsx b/apps/nextjs-app/src/features/app/components/sidebar/SidebarHeader.tsx
@@ -11,7 +11,7 @@ export const SidebarHeader = (prop: ISidebarHeaderProps) => {
   const { headerLeft, onExpand } = prop;
 
   return (
-    <div className="m-2 flex items-center gap-1">
+    <div className="m-2 flex h-7 items-center gap-1">
       {headerLeft}
       <div className="grow basis-0" />
       {onExpand && (