From add116cffc6e7b86da9b3ed1b1f77f381e394f38 Mon Sep 17 00:00:00 2001
From: moonrailgun <moonrailgun@gmail.com>
Date: Thu, 6 Apr 2023 10:48:54 +0800
Subject: [PATCH] perf: perf mobile view display in pagecontent

---
 .../src/routes/Main/Content/PageContent.tsx   | 25 +++++++++++--------
 1 file changed, 14 insertions(+), 11 deletions(-)

diff --git a/client/web/src/routes/Main/Content/PageContent.tsx b/client/web/src/routes/Main/Content/PageContent.tsx
index a4cc20d9..4d28ac82 100644
--- a/client/web/src/routes/Main/Content/PageContent.tsx
+++ b/client/web/src/routes/Main/Content/PageContent.tsx
@@ -1,4 +1,4 @@
-import React, { PropsWithChildren, useCallback, useEffect } from 'react';
+import React, { PropsWithChildren, useCallback } from 'react';
 import { useSidebarContext } from '../SidebarContext';
 import _isNil from 'lodash/isNil';
 import { EventTypes, useDrag, UserDragConfig } from '@use-gesture/react';
@@ -6,6 +6,7 @@ import { useIsMobile } from '@/hooks/useIsMobile';
 import clsx from 'clsx';
 import { ErrorBoundary } from '@/components/ErrorBoundary';
 import type { ReactDOMAttributes } from '@use-gesture/react/dist/declarations/src/types';
+import { useWatch } from 'tailchat-shared';
 
 interface PageContentRootProps extends PropsWithChildren<ReactDOMAttributes> {
   className?: string;
@@ -20,7 +21,7 @@ const PageContentRoot: React.FC<PageContentRootProps> = ({
   <div
     {...others}
     style={style}
-    className={clsx('flex flex-row flex-1 overflow-hidden', className)}
+    className={clsx('flex flex-row flex-1 overflow-hidden relative', className)}
   >
     {children}
   </div>
@@ -81,21 +82,17 @@ export const PageContent: React.FC<PropsWithChildren<PageContentProps>> =
       []
     );
 
-    useEffect(() => {
+    useWatch([isMobile], () => {
       if (isMobile === false) {
         // 如果不为移动端, 则一定显示侧边栏
         setShowSidebar(true);
       }
-    }, [isMobile]);
+    });
 
     const sidebarEl = _isNil(sidebar) ? null : (
       <div
         className={clsx(
-          'bg-sidebar-light dark:bg-sidebar-dark flex-shrink-0 transition-width',
-          {
-            'w-60': showSidebar,
-            'w-0': !showSidebar,
-          }
+          'bg-sidebar-light dark:bg-sidebar-dark flex-shrink-0 transition-width w-60'
         )}
       >
         {props.sidebar}
@@ -121,14 +118,20 @@ export const PageContent: React.FC<PropsWithChildren<PageContentProps>> =
 
         <div
           className={clsx(
-            'flex flex-auto bg-content-light dark:bg-content-dark relative overflow-hidden'
+            'flex flex-auto bg-content-light dark:bg-content-dark overflow-hidden z-10',
+            isMobile &&
+              'transform left-0 w-full h-full absolute transition-transform',
+            isMobile && {
+              'translate-x-60': showSidebar,
+              'translate-x-0': !showSidebar,
+            }
           )}
           data-tc-role={props['data-tc-role']}
         >
           <div className="tc-content-background" />
 
           <div
-            className={clsx('flex flex-auto relative', {
+            className={clsx('flex relative w-full', {
               'overflow-auto': !showMask,
               'overflow-hidden': showMask,
             })}