From 4c5191f0a0d1647ccc31d8ddf8737f93dab79370 Mon Sep 17 00:00:00 2001
From: amhsirak <karishmashuklaa@gmail.com>
Date: Wed, 5 Nov 2025 22:12:07 +0530
Subject: [PATCH] feat:  use sidebar layout for main app pages only

---
 src/pages/PageWrapper.tsx | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/src/pages/PageWrapper.tsx b/src/pages/PageWrapper.tsx
index 83004fcc..1af53681 100644
--- a/src/pages/PageWrapper.tsx
+++ b/src/pages/PageWrapper.tsx
@@ -85,26 +85,29 @@ export const PageWrapper = () => {
       }
     }
   }, [location.pathname, navigate, setBrowserId, setRecordingId, setRecordingName, setRecordingUrl]);
+
+  const isAuthPage = location.pathname === '/login' || location.pathname === '/register';
+  const isRecordingPage = location.pathname === '/recording';
   
   return (
     <div>
       <AuthProvider>
         <SocketProvider>
           <React.Fragment>
-            {/* Sticky NavBar - only show on non-recording pages */}
-            {location.pathname !== '/recording' && (
+            {/* Show NavBar only for main app pages, not for recording pages */}
+            {!isRecordingPage && (
               <Box sx={{ 
                 position: 'sticky', 
                 top: 0, 
                 zIndex: 1100,
-                backgroundColor: 'background.paper' 
+                backgroundColor: 'background.paper'
               }}>
                 <NavBar recordingName={recordingName} isRecording={false} />
               </Box>
             )}
             <Box sx={{ 
-              display: 'flex', 
-              minHeight: location.pathname !== '/recording' ? 'calc(100vh - 64px)' : '100vh' 
+              display: isAuthPage || isRecordingPage ? 'block' : 'flex', 
+              minHeight: isAuthPage || isRecordingPage ? '100vh' : 'calc(100vh - 64px)'
             }}>
               <Routes>
                 <Route element={<UserRoute />}>