From 2645a8e8ecf14d4f4785c30078e122e143b5f248 Mon Sep 17 00:00:00 2001
From: ZL Asica <40444637+ZL-Asica@users.noreply.github.com>
Date: Sun, 27 Oct 2024 23:44:00 -0500
Subject: [PATCH 1/4] refactor: move Footer out, use react router to get
 current page.

---
 src/App.css                      |  8 -------
 src/App.jsx                      | 27 +++-------------------
 src/components/common/Footer.jsx | 39 ++++++++++++++++++++++++++++++++
 3 files changed, 42 insertions(+), 32 deletions(-)
 create mode 100644 src/components/common/Footer.jsx

diff --git a/src/App.css b/src/App.css
index f270304..57d64b9 100644
--- a/src/App.css
+++ b/src/App.css
@@ -8,11 +8,3 @@
   flex: 1;
   padding-bottom: 56px; /* Space for the bottom navigation */
 }
-
-.bottom-nav {
-  position: fixed;
-  bottom: 0;
-  left: 0;
-  right: 0;
-  z-index: 10; /* Ensure it's above other elements */
-}
diff --git a/src/App.jsx b/src/App.jsx
index d8b15b9..1c0c50a 100644
--- a/src/App.jsx
+++ b/src/App.jsx
@@ -1,22 +1,16 @@
-import { useState } from 'react';
-import { Link, Route, BrowserRouter as Router, Routes } from 'react-router-dom';
+import { Route, BrowserRouter as Router, Routes } from 'react-router-dom';
 
-import FlagIcon from '@mui/icons-material/Flag';
-import HomeIcon from '@mui/icons-material/Home';
-import BottomNavigation from '@mui/material/BottomNavigation';
-import BottomNavigationAction from '@mui/material/BottomNavigationAction';
 import { ThemeProvider } from '@mui/material/styles';
 import { theme } from '@utils/theme';
 
 import Home from './pages/Home';
 
+import Footer from '@components/common/Footer';
 import Header from '@components/common/Header';
 import { UserProvider } from '@contexts/UserContext';
 import './App.css';
 
 const App = () => {
-  const [value, setValue] = useState(0);
-
   return (
     <ThemeProvider theme={theme}>
       <UserProvider>
@@ -32,22 +26,7 @@ const App = () => {
             </div>
 
             {/* Bottom Navigation with React Router links */}
-            <BottomNavigation
-              className="bottom-nav"
-              value={value}
-              onChange={(event, newValue) => {
-                setValue(newValue);
-              }}
-              showLabels
-            >
-              <BottomNavigationAction label="Home" icon={<HomeIcon />} component={Link} to="/" />
-              <BottomNavigationAction
-                label="Steak"
-                icon={<FlagIcon />}
-                component={Link}
-                to="/streak"
-              />
-            </BottomNavigation>
+            <Footer />
           </Router>
         </div>
       </UserProvider>
diff --git a/src/components/common/Footer.jsx b/src/components/common/Footer.jsx
new file mode 100644
index 0000000..2b12b84
--- /dev/null
+++ b/src/components/common/Footer.jsx
@@ -0,0 +1,39 @@
+import FlagIcon from '@mui/icons-material/Flag';
+import HomeIcon from '@mui/icons-material/Home';
+import { BottomNavigation, BottomNavigationAction } from '@mui/material';
+import { Link, useLocation } from 'react-router-dom';
+
+export default function Footer() {
+  const location = useLocation();
+
+  // Get the index of the page based on the path
+  const getPageIndex = (path) => {
+    switch (path) {
+      case '/':
+        return 0;
+      case '/streak':
+        return 1;
+      default:
+        return 0;
+    }
+  };
+
+  return (
+    <BottomNavigation
+      className="bottom-nav"
+      value={getPageIndex(location.pathname)}
+      showLabels
+      sx={{
+        backgroundColor: 'primary.light',
+        position: 'fixed',
+        bottom: 0,
+        left: 0,
+        right: 0,
+        zIndex: 1000, // Ensure the footer is always on top
+      }}
+    >
+      <BottomNavigationAction label="Home" icon={<HomeIcon />} component={Link} to="/" />
+      <BottomNavigationAction label="Streak" icon={<FlagIcon />} component={Link} to="/streak" />
+    </BottomNavigation>
+  );
+}

From 3ba591ae0367e145bbfb40c8d314c108478eb2e5 Mon Sep 17 00:00:00 2001
From: ZL Asica <40444637+ZL-Asica@users.noreply.github.com>
Date: Sun, 27 Oct 2024 23:44:40 -0500
Subject: [PATCH 2/4] refactor: Moved `Streak` out for future implementation.

---
 src/App.jsx          |  5 ++---
 src/pages/Streak.jsx | 11 +++++++++++
 2 files changed, 13 insertions(+), 3 deletions(-)
 create mode 100644 src/pages/Streak.jsx

diff --git a/src/App.jsx b/src/App.jsx
index 1c0c50a..30868c6 100644
--- a/src/App.jsx
+++ b/src/App.jsx
@@ -3,7 +3,8 @@ import { Route, BrowserRouter as Router, Routes } from 'react-router-dom';
 import { ThemeProvider } from '@mui/material/styles';
 import { theme } from '@utils/theme';
 
-import Home from './pages/Home';
+import Home from '@pages/Home';
+import Streak from '@pages/Streak';
 
 import Footer from '@components/common/Footer';
 import Header from '@components/common/Header';
@@ -35,5 +36,3 @@ const App = () => {
 };
 
 export default App;
-
-const Streak = () => <h1>Streak</h1>;
diff --git a/src/pages/Streak.jsx b/src/pages/Streak.jsx
new file mode 100644
index 0000000..e61349f
--- /dev/null
+++ b/src/pages/Streak.jsx
@@ -0,0 +1,11 @@
+import { Box } from '@mui/material';
+
+const Streak = () => {
+  return (
+    <Box sx={{ justifyContent: 'center', display: 'flex' }}>
+      <h1>Streak</h1>
+    </Box>
+  );
+};
+
+export default Streak;

From f4f27ab4876771f56e463270f64698813af9fcf3 Mon Sep 17 00:00:00 2001
From: ZL Asica <40444637+ZL-Asica@users.noreply.github.com>
Date: Sun, 27 Oct 2024 23:45:17 -0500
Subject: [PATCH 3/4] feat: ConfirmSignOutDialog to prevent accidental
 sign-outs.

---
 .../common/ConfirmSignOutDialog.jsx           | 26 ++++++++++
 src/components/common/Header.jsx              | 51 +++++++++++++------
 2 files changed, 61 insertions(+), 16 deletions(-)
 create mode 100644 src/components/common/ConfirmSignOutDialog.jsx

diff --git a/src/components/common/ConfirmSignOutDialog.jsx b/src/components/common/ConfirmSignOutDialog.jsx
new file mode 100644
index 0000000..244e618
--- /dev/null
+++ b/src/components/common/ConfirmSignOutDialog.jsx
@@ -0,0 +1,26 @@
+import { useUser } from '@contexts/UserContext';
+import { Button, Dialog, DialogActions, DialogTitle } from '@mui/material';
+
+const ConfirmSignOutDialog = ({ open, onClose }) => {
+  const { handleSignOut } = useUser();
+
+  // Function to confirm sign out
+  const confirmSignOut = async () => {
+    await handleSignOut();
+    onClose();
+  };
+
+  return (
+    <Dialog open={open} onClose={onClose}>
+      <DialogTitle>Confirm Sign Out</DialogTitle>
+      <DialogActions>
+        <Button onClick={onClose}>Cancel</Button>
+        <Button onClick={confirmSignOut} color="error">
+          Sign Out
+        </Button>
+      </DialogActions>
+    </Dialog>
+  );
+};
+
+export default ConfirmSignOutDialog;
diff --git a/src/components/common/Header.jsx b/src/components/common/Header.jsx
index 60a7c8c..e2d4735 100644
--- a/src/components/common/Header.jsx
+++ b/src/components/common/Header.jsx
@@ -1,23 +1,34 @@
 import { useUser } from '@contexts/UserContext';
+import ArrowBackIcon from '@mui/icons-material/ArrowBack';
 import { AppBar, Avatar, Box, Button, IconButton, Toolbar, Typography } from '@mui/material';
+import { useState } from 'react';
+import { useLocation, useNavigate } from 'react-router-dom';
+import ConfirmSignOutDialog from './ConfirmSignOutDialog';
 
 const Header = () => {
-  const { user, handleSignIn, handleSignOut } = useUser();
+  const { user, handleSignIn } = useUser();
+  const location = useLocation();
+  const navigate = useNavigate();
 
-  const handleAuthClick = async () => {
-    if (user) {
-      await handleSignOut();
-    } else {
-      await handleSignIn();
-    }
-  };
+  // State for Dialog visibility
+  const [openConfirmDialog, setOpenConfirmDialog] = useState(false);
+
+  // Show back button only on pages other than Home and Streak
+  const showBackButton = location.pathname !== '/' && location.pathname !== '/streak';
 
   return (
     <AppBar position="sticky" sx={{ backgroundColor: 'primary.light', color: '#000' }}>
-      <Toolbar sx={{ position: 'relative', justifyContent: 'space-between' }}>
+      <Toolbar sx={{ justifyContent: 'space-between', position: 'relative' }}>
         <Box sx={{ display: 'flex', alignItems: 'center' }}>
-          {/* Placeholder Box for back button if needed in the future */}
-          <Box sx={{ width: '48px' }} />
+          {showBackButton && (
+            <IconButton
+              edge="start"
+              color="inherit"
+              onClick={() => navigate(-1)} // Back button
+            >
+              <ArrowBackIcon />
+            </IconButton>
+          )}
         </Box>
 
         <Typography
@@ -26,7 +37,7 @@ const Header = () => {
             position: 'absolute',
             left: '50%',
             transform: 'translateX(-50%)',
-            fontWeight: '600',
+            fontWeight: 600,
             fontSize: '1.4rem',
           }}
         >
@@ -35,11 +46,19 @@ const Header = () => {
 
         <Box sx={{ display: 'flex', alignItems: 'center' }}>
           {user ? (
-            <IconButton edge="end" color="inherit" onClick={handleAuthClick}>
-              <Avatar alt={user.displayName} src={user.photoURL} />
-            </IconButton>
+            <>
+              <IconButton edge="end" color="inherit" onClick={() => setOpenConfirmDialog(true)}>
+                <Avatar alt={user.displayName} src={user.photoURL} />
+              </IconButton>
+
+              {/* Dialog for Confirm Sign Out */}
+              <ConfirmSignOutDialog
+                open={openConfirmDialog}
+                onClose={() => setOpenConfirmDialog(false)}
+              />
+            </>
           ) : (
-            <Button color="inherit" onClick={handleAuthClick}>
+            <Button color="inherit" onClick={handleSignIn}>
               Sign In
             </Button>
           )}

From d5221b2291939190bacbaa5990d5fda6a21f6687 Mon Sep 17 00:00:00 2001
From: ZL Asica <40444637+ZL-Asica@users.noreply.github.com>
Date: Sun, 27 Oct 2024 23:45:39 -0500
Subject: [PATCH 4/4] fix : Header position should outside of the content div.

---
 src/App.jsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/App.jsx b/src/App.jsx
index 30868c6..aa854a6 100644
--- a/src/App.jsx
+++ b/src/App.jsx
@@ -17,8 +17,8 @@ const App = () => {
       <UserProvider>
         <div className="App">
           <Router>
+            <Header />
             <div className="content">
-              <Header />
               {/* Main content area where pages will render */}
               <Routes>
                 <Route path="/" element={<Home />} />