From dd7b0896327c1b36f0bc1f071a6efd9b8da699bc Mon Sep 17 00:00:00 2001
From: Mark Volkmann <r.mark.volkmann@gmail.com>
Date: Thu, 25 Apr 2024 11:53:27 -0500
Subject: [PATCH 1/5] initial work on removing avatar menu

---
 web-ui/src/components/menu/Menu.jsx | 44 ++---------------------------
 1 file changed, 2 insertions(+), 42 deletions(-)

diff --git a/web-ui/src/components/menu/Menu.jsx b/web-ui/src/components/menu/Menu.jsx
index 33c7dae9ee..31ed098439 100644
--- a/web-ui/src/components/menu/Menu.jsx
+++ b/web-ui/src/components/menu/Menu.jsx
@@ -20,7 +20,6 @@ import fileDownload from 'js-file-download';
 import { useLocation, Link } from 'react-router-dom';
 import { AppContext } from '../../context/AppContext';
 import { getAvatarURL } from '../../api/api';
-import AvatarMenu from '@mui/material/Menu';
 
 import MenuIcon from '@mui/icons-material/Menu';
 import { styled, useTheme } from '@mui/material/styles';
@@ -183,8 +182,8 @@ function Menu() {
     return links;
   };
 
-  const handleClick = event => {
-    setAnchorEl(event.currentTarget);
+  const handleClick = () => {
+    location.href = `/profile/${id}`;
   };
 
   const downloadMembers = async () => {
@@ -304,10 +303,6 @@ function Menu() {
     setFeedbackOpen(false);
   };
 
-  const closeAvatarMenu = () => {
-    setAnchorEl(null);
-  };
-
   const closeHoursUpload = () => {
     setShowHoursUpload(false);
     setSelectedFile(null);
@@ -447,41 +442,6 @@ function Menu() {
               textDecoration: 'none'
             }}
           />
-          <AvatarMenu
-            id="simple-menu"
-            anchorEl={anchorEl}
-            keepMounted
-            open={Boolean(anchorEl)}
-            onClose={closeAvatarMenu}
-          >
-            <MenuItem
-              component={Link}
-              onClick={closeAvatarMenu}
-              to={`/profile/${id}`}
-            >
-              Profile
-            </MenuItem>
-            {isAdmin && (
-              <MenuItem
-                onClick={() => {
-                  closeAvatarMenu();
-                  openHoursUpload();
-                }}
-              >
-                Upload Hours
-              </MenuItem>
-            )}
-            {isAdmin && (
-              <MenuItem
-                onClick={() => {
-                  closeAvatarMenu();
-                  downloadMembers();
-                }}
-              >
-                Download Members
-              </MenuItem>
-            )}
-          </AvatarMenu>
         </div>
       </AppBar>
       <nav className={classes.drawer}>

From f8bf3c0cb88e0148e654e484896b809811deae87 Mon Sep 17 00:00:00 2001
From: Mark Volkmann <r.mark.volkmann@gmail.com>
Date: Thu, 25 Apr 2024 12:04:51 -0500
Subject: [PATCH 2/5] progress

---
 web-ui/src/components/menu/Menu.jsx | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/web-ui/src/components/menu/Menu.jsx b/web-ui/src/components/menu/Menu.jsx
index 31ed098439..9c642ee3b1 100644
--- a/web-ui/src/components/menu/Menu.jsx
+++ b/web-ui/src/components/menu/Menu.jsx
@@ -182,7 +182,8 @@ function Menu() {
     return links;
   };
 
-  const handleClick = () => {
+  const handleAvatarClick = () => {
+    console.log('Menu.jsx handleAvatarClick: id =', id);
     location.href = `/profile/${id}`;
   };
 
@@ -432,7 +433,7 @@ function Menu() {
           onClick={handleToggle}
         >
           <Avatar
-            onClick={handleClick}
+            onClick={handleAvatarClick}
             src={getAvatarURL(workEmail)}
             style={{
               position: 'absolute',

From 22b12e53afeb789c42bf7ad7e983768cb6cb292f Mon Sep 17 00:00:00 2001
From: Mark Volkmann <r.mark.volkmann@gmail.com>
Date: Thu, 25 Apr 2024 12:15:29 -0500
Subject: [PATCH 3/5] changed avatar to go to profile page on click

---
 web-ui/src/components/menu/Menu.jsx | 16 ++--------------
 1 file changed, 2 insertions(+), 14 deletions(-)

diff --git a/web-ui/src/components/menu/Menu.jsx b/web-ui/src/components/menu/Menu.jsx
index 9c642ee3b1..93f84f83b9 100644
--- a/web-ui/src/components/menu/Menu.jsx
+++ b/web-ui/src/components/menu/Menu.jsx
@@ -34,7 +34,6 @@ import {
   List,
   ListItem,
   ListItemText,
-  MenuItem,
   Modal,
   Toolbar
 } from '@mui/material';
@@ -182,11 +181,6 @@ function Menu() {
     return links;
   };
 
-  const handleAvatarClick = () => {
-    console.log('Menu.jsx handleAvatarClick: id =', id);
-    location.href = `/profile/${id}`;
-  };
-
   const downloadMembers = async () => {
     let res = await reportAllMembersCsv(csrf);
     if (res?.error) {
@@ -426,14 +420,8 @@ function Menu() {
             <MenuIcon />
           </IconButton>
         </Toolbar>
-        <div
-          ref={anchorRef}
-          aria-controls={open ? 'menu-list-grow' : undefined}
-          aria-haspopup="true"
-          onClick={handleToggle}
-        >
+        <Link to={`/profile/${id}`}>
           <Avatar
-            onClick={handleAvatarClick}
             src={getAvatarURL(workEmail)}
             style={{
               position: 'absolute',
@@ -443,7 +431,7 @@ function Menu() {
               textDecoration: 'none'
             }}
           />
-        </div>
+        </Link>
       </AppBar>
       <nav className={classes.drawer}>
         <Drawer

From 26bd09d86a0e49cfe669c9819ddede1c6231daf1 Mon Sep 17 00:00:00 2001
From: Mark Volkmann <r.mark.volkmann@gmail.com>
Date: Thu, 25 Apr 2024 12:19:19 -0500
Subject: [PATCH 4/5] updated snapshots related to the old avatar menu

---
 .../menu/__snapshots__/Menu.test.jsx.snap      | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/web-ui/src/components/menu/__snapshots__/Menu.test.jsx.snap b/web-ui/src/components/menu/__snapshots__/Menu.test.jsx.snap
index a4581bc86d..22ff366193 100644
--- a/web-ui/src/components/menu/__snapshots__/Menu.test.jsx.snap
+++ b/web-ui/src/components/menu/__snapshots__/Menu.test.jsx.snap
@@ -33,8 +33,8 @@ exports[`<Menu /> > renders correctly 1`] = `
           />
         </button>
       </div>
-      <div
-        aria-haspopup="true"
+      <a
+        href="/profile/undefined"
       >
         <div
           class="MuiAvatar-root MuiAvatar-circular css-1wlk0hk-MuiAvatar-root"
@@ -45,7 +45,7 @@ exports[`<Menu /> > renders correctly 1`] = `
             src="http://localhost:8080/services/member-profiles/member-photos/test%40tester.com"
           />
         </div>
-      </div>
+      </a>
     </header>
     <nav
       class="Menu-drawer"
@@ -436,8 +436,8 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
           />
         </button>
       </div>
-      <div
-        aria-haspopup="true"
+      <a
+        href="/profile/undefined"
       >
         <div
           class="MuiAvatar-root MuiAvatar-circular css-1wlk0hk-MuiAvatar-root"
@@ -448,7 +448,7 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
             src="http://localhost:8080/services/member-profiles/member-photos/test%40tester.com"
           />
         </div>
-      </div>
+      </a>
     </header>
     <nav
       class="Menu-drawer"
@@ -911,8 +911,8 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
           />
         </button>
       </div>
-      <div
-        aria-haspopup="true"
+      <a
+        href="/profile/undefined"
       >
         <div
           class="MuiAvatar-root MuiAvatar-circular css-1wlk0hk-MuiAvatar-root"
@@ -923,7 +923,7 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
             src="http://localhost:8080/services/member-profiles/member-photos/test%40tester.com"
           />
         </div>
-      </div>
+      </a>
     </header>
     <nav
       class="Menu-drawer"

From 8983a7eae6d05a6bfe9832fb175c273a41f392b9 Mon Sep 17 00:00:00 2001
From: Mark Volkmann <r.mark.volkmann@gmail.com>
Date: Fri, 26 Apr 2024 09:16:02 -0500
Subject: [PATCH 5/5] added a test in Menu.test.jsx

---
 .../edit_skills/EditSkillsCard.test.jsx       |   1 -
 web-ui/src/components/menu/Menu.jsx           |   2 -
 web-ui/src/components/menu/Menu.test.jsx      |  19 ++
 .../menu/__snapshots__/Menu.test.jsx.snap     | 182 +++---------------
 4 files changed, 50 insertions(+), 154 deletions(-)

diff --git a/web-ui/src/components/edit_skills/EditSkillsCard.test.jsx b/web-ui/src/components/edit_skills/EditSkillsCard.test.jsx
index 7cf0f15592..5260dc1607 100644
--- a/web-ui/src/components/edit_skills/EditSkillsCard.test.jsx
+++ b/web-ui/src/components/edit_skills/EditSkillsCard.test.jsx
@@ -5,7 +5,6 @@ import { AppContextProvider } from '../../context/AppContext';
 import { http, HttpResponse } from 'msw';
 import { setupServer } from 'msw/node';
 import { BrowserRouter } from 'react-router-dom';
-import { act } from '@testing-library/react';
 
 const currentUserProfile = {
   id: 9876,
diff --git a/web-ui/src/components/menu/Menu.jsx b/web-ui/src/components/menu/Menu.jsx
index 0e78e81cc6..0990f309cf 100644
--- a/web-ui/src/components/menu/Menu.jsx
+++ b/web-ui/src/components/menu/Menu.jsx
@@ -147,7 +147,6 @@ function Menu() {
 
   const [mobileOpen, setMobileOpen] = useState(false);
   const [open, setOpen] = useState(false);
-  const [anchorEl, setAnchorEl] = useState(null);
   const [showHoursUpload, setShowHoursUpload] = useState(false);
   const [selectedFile, setSelectedFile] = useState(null);
   const feedbackLinks = getFeedbackLinks(isAdmin, isPDL, isSupervisor);
@@ -295,7 +294,6 @@ function Menu() {
         component={Link}
         to={path}
         className={isSubLink ? classes.nested : null}
-        button
         onClick={
           isSubLink
             ? undefined
diff --git a/web-ui/src/components/menu/Menu.test.jsx b/web-ui/src/components/menu/Menu.test.jsx
index 9aebc6746c..fe38cf1a75 100644
--- a/web-ui/src/components/menu/Menu.test.jsx
+++ b/web-ui/src/components/menu/Menu.test.jsx
@@ -1,13 +1,17 @@
 import React from 'react';
+import { render } from '@testing-library/react';
 import Menu from './Menu';
+import { BrowserRouter } from 'react-router-dom';
 import { MemoryRouter } from 'react-router-dom';
 import { AppContextProvider } from '../../context/AppContext';
 
+const testId = 'some-id';
 const initialState = {
   state: {
     userProfile: {
       name: 'holmes',
       memberProfile: {
+        id: testId,
         pdlId: '',
         title: 'Tester',
         workEmail: 'test@tester.com'
@@ -84,4 +88,19 @@ describe('<Menu />', () => {
       </AppContextProvider>
     );
   });
+
+  it('adds link to avatar', () => {
+    const { container } = render(
+      <AppContextProvider value={initialState}>
+        <BrowserRouter>
+          <Menu />
+        </BrowserRouter>
+      </AppContextProvider>
+    );
+    const link = container.querySelector('header > a');
+    const href = link.getAttribute('href');
+    const lastIndex = href.lastIndexOf('/');
+    const id = href.substring(lastIndex + 1);
+    expect(id).toBe(testId);
+  });
 });
diff --git a/web-ui/src/components/menu/__snapshots__/Menu.test.jsx.snap b/web-ui/src/components/menu/__snapshots__/Menu.test.jsx.snap
index 3bcb624969..483bf04f81 100644
--- a/web-ui/src/components/menu/__snapshots__/Menu.test.jsx.snap
+++ b/web-ui/src/components/menu/__snapshots__/Menu.test.jsx.snap
@@ -34,7 +34,7 @@ exports[`<Menu /> > renders correctly 1`] = `
         </button>
       </div>
       <a
-        href="/profile/undefined"
+        href="/profile/some-id"
       >
         <div
           class="MuiAvatar-root MuiAvatar-circular css-1wlk0hk-MuiAvatar-root"
@@ -84,9 +84,8 @@ exports[`<Menu /> > renders correctly 1`] = `
               class="MuiList-root MuiList-padding Menu-listStyle css-h4y409-MuiList-root"
             >
               <a
-                class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
                 href="/"
-                tabindex="0"
               >
                 <div
                   class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -97,14 +96,10 @@ exports[`<Menu /> > renders correctly 1`] = `
                     HOME
                   </span>
                 </div>
-                <span
-                  class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                />
               </a>
               <a
-                class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
                 href="/checkins"
-                tabindex="0"
               >
                 <div
                   class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -115,9 +110,6 @@ exports[`<Menu /> > renders correctly 1`] = `
                     CHECK-INS
                   </span>
                 </div>
-                <span
-                  class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                />
               </a>
               <li
                 class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
@@ -143,9 +135,8 @@ exports[`<Menu /> > renders correctly 1`] = `
                     class="MuiCollapse-wrapperInner MuiCollapse-vertical css-9l5vo-MuiCollapse-wrapperInner"
                   >
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Mui-selected Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Mui-selected Menu-nested css-1p823my-MuiListItem-root"
                       href="/guilds"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -156,14 +147,10 @@ exports[`<Menu /> > renders correctly 1`] = `
                           Guilds
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Menu-nested css-1p823my-MuiListItem-root"
                       href="/people"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -174,14 +161,10 @@ exports[`<Menu /> > renders correctly 1`] = `
                           People
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Menu-nested css-1p823my-MuiListItem-root"
                       href="/teams"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -192,9 +175,6 @@ exports[`<Menu /> > renders correctly 1`] = `
                           Teams
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                   </div>
                 </div>
@@ -241,9 +221,8 @@ exports[`<Menu /> > renders correctly 1`] = `
               class="MuiList-root MuiList-padding Menu-listStyle css-h4y409-MuiList-root"
             >
               <a
-                class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
                 href="/"
-                tabindex="0"
               >
                 <div
                   class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -254,14 +233,10 @@ exports[`<Menu /> > renders correctly 1`] = `
                     HOME
                   </span>
                 </div>
-                <span
-                  class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                />
               </a>
               <a
-                class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
                 href="/checkins"
-                tabindex="0"
               >
                 <div
                   class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -272,9 +247,6 @@ exports[`<Menu /> > renders correctly 1`] = `
                     CHECK-INS
                   </span>
                 </div>
-                <span
-                  class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                />
               </a>
               <li
                 class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
@@ -300,9 +272,8 @@ exports[`<Menu /> > renders correctly 1`] = `
                     class="MuiCollapse-wrapperInner MuiCollapse-vertical css-9l5vo-MuiCollapse-wrapperInner"
                   >
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Mui-selected Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Mui-selected Menu-nested css-1p823my-MuiListItem-root"
                       href="/guilds"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -313,14 +284,10 @@ exports[`<Menu /> > renders correctly 1`] = `
                           Guilds
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Menu-nested css-1p823my-MuiListItem-root"
                       href="/people"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -331,14 +298,10 @@ exports[`<Menu /> > renders correctly 1`] = `
                           People
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Menu-nested css-1p823my-MuiListItem-root"
                       href="/teams"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -349,9 +312,6 @@ exports[`<Menu /> > renders correctly 1`] = `
                           Teams
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                   </div>
                 </div>
@@ -467,9 +427,8 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
               class="MuiList-root MuiList-padding Menu-listStyle css-h4y409-MuiList-root"
             >
               <a
-                class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
                 href="/"
-                tabindex="0"
               >
                 <div
                   class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -480,9 +439,6 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
                     HOME
                   </span>
                 </div>
-                <span
-                  class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                />
               </a>
               <li
                 class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
@@ -498,9 +454,8 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
                 </div>
               </li>
               <a
-                class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
                 href="/checkins"
-                tabindex="0"
               >
                 <div
                   class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -511,9 +466,6 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
                     CHECK-INS
                   </span>
                 </div>
-                <span
-                  class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                />
               </a>
               <li
                 class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
@@ -539,9 +491,8 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
                     class="MuiCollapse-wrapperInner MuiCollapse-vertical css-9l5vo-MuiCollapse-wrapperInner"
                   >
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Mui-selected Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Mui-selected Menu-nested css-1p823my-MuiListItem-root"
                       href="/guilds"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -552,14 +503,10 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
                           Guilds
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Menu-nested css-1p823my-MuiListItem-root"
                       href="/people"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -570,14 +517,10 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
                           People
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Menu-nested css-1p823my-MuiListItem-root"
                       href="/teams"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -588,9 +531,6 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
                           Teams
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                   </div>
                 </div>
@@ -655,9 +595,8 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
               class="MuiList-root MuiList-padding Menu-listStyle css-h4y409-MuiList-root"
             >
               <a
-                class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
                 href="/"
-                tabindex="0"
               >
                 <div
                   class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -668,9 +607,6 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
                     HOME
                   </span>
                 </div>
-                <span
-                  class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                />
               </a>
               <li
                 class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
@@ -686,9 +622,8 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
                 </div>
               </li>
               <a
-                class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
                 href="/checkins"
-                tabindex="0"
               >
                 <div
                   class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -699,9 +634,6 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
                     CHECK-INS
                   </span>
                 </div>
-                <span
-                  class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                />
               </a>
               <li
                 class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
@@ -727,9 +659,8 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
                     class="MuiCollapse-wrapperInner MuiCollapse-vertical css-9l5vo-MuiCollapse-wrapperInner"
                   >
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Mui-selected Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Mui-selected Menu-nested css-1p823my-MuiListItem-root"
                       href="/guilds"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -740,14 +671,10 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
                           Guilds
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Menu-nested css-1p823my-MuiListItem-root"
                       href="/people"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -758,14 +685,10 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
                           People
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Menu-nested css-1p823my-MuiListItem-root"
                       href="/teams"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -776,9 +699,6 @@ exports[`<Menu /> > renders correctly for admin 1`] = `
                           Teams
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                   </div>
                 </div>
@@ -912,9 +832,8 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
               class="MuiList-root MuiList-padding Menu-listStyle css-h4y409-MuiList-root"
             >
               <a
-                class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
                 href="/"
-                tabindex="0"
               >
                 <div
                   class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -925,14 +844,10 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
                     HOME
                   </span>
                 </div>
-                <span
-                  class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                />
               </a>
               <a
-                class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
                 href="/checkins"
-                tabindex="0"
               >
                 <div
                   class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -943,9 +858,6 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
                     CHECK-INS
                   </span>
                 </div>
-                <span
-                  class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                />
               </a>
               <li
                 class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
@@ -971,9 +883,8 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
                     class="MuiCollapse-wrapperInner MuiCollapse-vertical css-9l5vo-MuiCollapse-wrapperInner"
                   >
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Mui-selected Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Mui-selected Menu-nested css-1p823my-MuiListItem-root"
                       href="/guilds"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -984,14 +895,10 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
                           Guilds
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Menu-nested css-1p823my-MuiListItem-root"
                       href="/people"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -1002,14 +909,10 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
                           People
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Menu-nested css-1p823my-MuiListItem-root"
                       href="/teams"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -1020,9 +923,6 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
                           Teams
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                   </div>
                 </div>
@@ -1069,9 +969,8 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
               class="MuiList-root MuiList-padding Menu-listStyle css-h4y409-MuiList-root"
             >
               <a
-                class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
                 href="/"
-                tabindex="0"
               >
                 <div
                   class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -1082,14 +981,10 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
                     HOME
                   </span>
                 </div>
-                <span
-                  class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                />
               </a>
               <a
-                class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
                 href="/checkins"
-                tabindex="0"
               >
                 <div
                   class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -1100,9 +995,6 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
                     CHECK-INS
                   </span>
                 </div>
-                <span
-                  class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                />
               </a>
               <li
                 class="MuiListItem-root MuiListItem-gutters MuiListItem-padding css-1p823my-MuiListItem-root"
@@ -1128,9 +1020,8 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
                     class="MuiCollapse-wrapperInner MuiCollapse-vertical css-9l5vo-MuiCollapse-wrapperInner"
                   >
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Mui-selected Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Mui-selected Menu-nested css-1p823my-MuiListItem-root"
                       href="/guilds"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -1141,14 +1032,10 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
                           Guilds
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Menu-nested css-1p823my-MuiListItem-root"
                       href="/people"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -1159,14 +1046,10 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
                           People
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                     <a
-                      class="MuiButtonBase-root MuiListItem-root MuiListItem-gutters MuiListItem-padding MuiListItem-button Menu-nested css-bshv44-MuiButtonBase-root-MuiListItem-root"
+                      class="MuiListItem-root MuiListItem-gutters MuiListItem-padding Menu-nested css-1p823my-MuiListItem-root"
                       href="/teams"
-                      tabindex="0"
                     >
                       <div
                         class="MuiListItemText-root css-tlelie-MuiListItemText-root"
@@ -1177,9 +1060,6 @@ exports[`<Menu /> > renders correctly for pdl 1`] = `
                           Teams
                         </span>
                       </div>
-                      <span
-                        class="MuiTouchRipple-root css-8je8zh-MuiTouchRipple-root"
-                      />
                     </a>
                   </div>
                 </div>