From 84f8ecbb7283f5b61caf35ea302f9fab33ca223c Mon Sep 17 00:00:00 2001
From: Anto Subash <antosubash@live.com>
Date: Mon, 25 May 2026 22:27:42 +0200
Subject: [PATCH] fix(ui): add accessible names to sidebar toggle and close
 buttons (#172)

The hamburger (open) and X (close) buttons in SidebarLayout had no
accessible names, making them invisible to screen readers. Add
aria-label="Open sidebar" and aria-label="Close sidebar" respectively.
---
 packages/ui/src/layouts/SidebarLayout.tsx | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/packages/ui/src/layouts/SidebarLayout.tsx b/packages/ui/src/layouts/SidebarLayout.tsx
index 93e7ed35..b957a0c7 100644
--- a/packages/ui/src/layouts/SidebarLayout.tsx
+++ b/packages/ui/src/layouts/SidebarLayout.tsx
@@ -82,6 +82,7 @@ export function SidebarLayout({
             variant="ghost"
             size="icon-sm"
             onClick={() => setSidebarOpen(true)}
+            aria-label="Open sidebar"
             className="text-sidebar-icon hover:text-white hover:bg-white/10"
           >
             <svg
@@ -141,6 +142,7 @@ export function SidebarLayout({
               variant="ghost"
               size="icon-sm"
               onClick={closeSidebar}
+              aria-label="Close sidebar"
               className="lg:hidden text-sidebar-icon-muted hover:text-white hover:bg-white/10"
             >
               <svg