From 88bf525ae4ee866e5069320cda34494bfdc19606 Mon Sep 17 00:00:00 2001
From: Marco Pedone <m.pedone98@gmail.com>
Date: Mon, 17 Nov 2025 17:29:47 +0100
Subject: [PATCH] refactor: fix sidebar expand functionality and improve button
 accessibility

---
 .../src/components/area-riservata/sidebar.tsx | 25 +++++++++++--------
 1 file changed, 14 insertions(+), 11 deletions(-)

diff --git a/apps/infoalloggi/src/components/area-riservata/sidebar.tsx b/apps/infoalloggi/src/components/area-riservata/sidebar.tsx
index be7938b..2c2c909 100644
--- a/apps/infoalloggi/src/components/area-riservata/sidebar.tsx
+++ b/apps/infoalloggi/src/components/area-riservata/sidebar.tsx
@@ -33,26 +33,29 @@ export const Sidebar = ({
 			setIsAnimating(false);
 		}, 300);
 	};
-	//todo click to expand not working on direct links when minimized
+
 	return (
 		<motion.nav
 			animate={{ width: minimized ? 55 : 192 }}
 			aria-label="Sidebar Menu"
 			className={cn(
-				"z-30 hidden h-auto w-48 shrink-0 space-y-4 border-muted border-r bg-background pt-3 md:block",
+				"relative z-30 hidden h-auto w-48 shrink-0 space-y-4 border-muted border-r bg-background pt-3 md:block",
 				className,
 				minimized && "cursor-pointer",
 			)}
-			onClick={
-				minimized
-					? (e) => {
-							e.preventDefault();
-							e.stopPropagation();
-							toggleSidebar(false);
-						}
-					: undefined
-			}
 		>
+			{minimized && (
+				<button
+					aria-label="Expand sidebar"
+					className="absolute inset-0 z-50 cursor-pointer border-none bg-transparent p-0 hover:bg-black/2"
+					onClick={(e) => {
+						e.preventDefault();
+						e.stopPropagation();
+						toggleSidebar(false);
+					}}
+					type="button"
+				/>
+			)}
 			<div
 				className={cn(
 					"flex h-8 items-center justify-between px-4",