From 62103ea752df1aff0a1ca35b9eaef420dd820b76 Mon Sep 17 00:00:00 2001
From: Richard R <me@richardr.dev>
Date: Mon, 4 May 2026 15:23:48 -0600
Subject: [PATCH] style(ui): update aria-labels for clarity and consistency in
 sidebar controls

Standardize aria-labels in DocumentHeaderMenu and ReaderSidebarShell to use
"Hide" and "Close" phrasing for improved accessibility and user experience.
---
 src/components/documents/DocumentHeaderMenu.tsx | 6 +++---
 src/components/reader/ReaderSidebarShell.tsx    | 4 ++--
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/components/documents/DocumentHeaderMenu.tsx b/src/components/documents/DocumentHeaderMenu.tsx
index 706c7be..fc12776 100644
--- a/src/components/documents/DocumentHeaderMenu.tsx
+++ b/src/components/documents/DocumentHeaderMenu.tsx
@@ -54,7 +54,7 @@ export function DocumentHeaderMenu({
               ? 'border-accent text-accent bg-offbase'
               : 'border-offbase text-foreground hover:bg-offbase hover:text-accent'
           }`}
-          aria-label={isSegmentsOpen ? 'Close segments sidebar' : 'Open segments sidebar'}
+          aria-label={isSegmentsOpen ? 'Hide segments sidebar' : 'Open segments sidebar'}
           title={isSegmentsOpen ? 'Hide Segments' : 'Segments'}
         >
           <ListIcon className="w-4 h-4 transform transition-transform duration-200 ease-in-out hover:scale-[1.09]" />
@@ -68,7 +68,7 @@ export function DocumentHeaderMenu({
               ? 'border-accent text-accent bg-offbase'
               : 'border-offbase text-foreground hover:bg-offbase hover:text-accent'
           }`}
-          aria-label={isAudiobookOpen ? 'Close audiobook export' : 'Open audiobook export'}
+          aria-label={isAudiobookOpen ? 'Hide audiobook export' : 'Open audiobook export'}
           title={isAudiobookOpen ? 'Hide Export Audiobook' : 'Export Audiobook'}
         >
           <DownloadIcon className="w-4 h-4 transform transition-transform duration-200 ease-in-out hover:scale-[1.09]" />
@@ -81,7 +81,7 @@ export function DocumentHeaderMenu({
             ? 'border-accent text-accent bg-offbase'
             : 'border-offbase text-foreground hover:bg-offbase hover:text-accent'
         }`}
-        aria-label={isSettingsOpen ? 'Close settings' : 'Open settings'}
+        aria-label={isSettingsOpen ? 'Hide settings' : 'Open settings'}
         title={isSettingsOpen ? 'Hide Settings' : 'Settings'}
       >
         <FileSettingsIcon className="w-4 h-4 transform transition-transform duration-200 ease-in-out hover:scale-[1.09]" />
diff --git a/src/components/reader/ReaderSidebarShell.tsx b/src/components/reader/ReaderSidebarShell.tsx
index ecb0c42..dbc9837 100644
--- a/src/components/reader/ReaderSidebarShell.tsx
+++ b/src/components/reader/ReaderSidebarShell.tsx
@@ -47,7 +47,7 @@ export function ReaderSidebarShell({
         >
           <button
             type="button"
-            aria-label={`Close ${title.toLowerCase()}`}
+            aria-label="Close"
             onClick={onClose}
             className="sm:hidden absolute inset-0 overlay-dim backdrop-blur-sm pointer-events-auto"
           />
@@ -76,7 +76,7 @@ export function ReaderSidebarShell({
                 <button
                   type="button"
                   onClick={onClose}
-                  aria-label={`Close ${title.toLowerCase()}`}
+                  aria-label="Close"
                   title="Close"
                   className="inline-flex items-center justify-center w-8 h-8 rounded-lg border border-offbase bg-base text-muted hover:bg-offbase hover:text-accent transition-colors"
                 >