From 30881a0af1c9d4e7817a0b18bb5b60b7e6c08682 Mon Sep 17 00:00:00 2001
From: gorkem-bwl <gorkem.cetin@bluewavelabs.ca>
Date: Fri, 28 Aug 2026 19:28:29 -0400
Subject: [PATCH] fix(client): drop invalid li wrapper from sidebar groups
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The group wrapper was rendering as `<li>`, but the enclosing `List` is
`component="nav"` and so renders a `<nav>`, not a `<ul>`. That put list items
directly inside a nav — invalid HTML, and a screen reader announces orphaned
list items with no enclosing list or item count.

`NavItem` renders a `div` via `ListItemButton`, so there is no list structure
for the group wrapper to join. Makes it a plain `Box` and drops the
now-redundant `listStyle: "none"`.

Claude-Session: https://claude.ai/code/session_01AbYLvwpEaZFXktHVcqrFRd
---
 client/src/Components/sidebar/index.tsx | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/client/src/Components/sidebar/index.tsx b/client/src/Components/sidebar/index.tsx
index 6c7bdfb78..1d1257414 100644
--- a/client/src/Components/sidebar/index.tsx
+++ b/client/src/Components/sidebar/index.tsx
@@ -92,11 +92,7 @@ export const Sidebar = () => {
 						pb={theme.spacing(LAYOUT.MD)}
 					/>
 					{menu.map((group, groupIndex) => (
-						<Box
-							component="li"
-							key={group.group}
-							sx={{ listStyle: "none" }}
-						>
+						<Box key={group.group}>
 							{collapsed ? (
 								groupIndex > 0 && (
 									<Divider