From a755a0b3f97e91d64b723c580d6f9f61092e1edb Mon Sep 17 00:00:00 2001
From: Richard Roberson <me@richardr.dev>
Date: Sat, 22 Nov 2025 15:27:04 -0700
Subject: [PATCH] refactor(ui): make UI labels and descriptions more concise

---
 src/app/page.tsx                 |  2 +-
 src/components/SettingsModal.tsx | 12 ++++++------
 tests/helpers.ts                 |  2 +-
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/app/page.tsx b/src/app/page.tsx
index 24a4558..569a797 100644
--- a/src/app/page.tsx
+++ b/src/app/page.tsx
@@ -11,7 +11,7 @@ export default function Home() {
         <div className="max-w-5xl mx-auto">
           <h1 className="text-2xl md:text-3xl font-bold tracking-tight mb-2 text-foreground">OpenReader WebUI</h1>
           <p className="text-sm leading-relaxed max-w-[77ch] text-foreground">
-            Open source document reader web app {isDev ? 'self-hosted server' : 'demo'}.
+            Open source document reader {isDev ? 'self-hosted server' : 'demo app'}.
             <span className="block font-medium">Read & listen to PDF, EPUB, MD, and TXT documents with high quality text to speech voices.</span>
           </p>
         </div>
diff --git a/src/components/SettingsModal.tsx b/src/components/SettingsModal.tsx
index 245bd89..ea5a770 100644
--- a/src/components/SettingsModal.tsx
+++ b/src/components/SettingsModal.tsx
@@ -599,7 +599,7 @@ export function SettingsModal() {
 
                       <TabPanel className="space-y-4">
                         {isDev && <div className="space-y-1">
-                          <label className="block text-sm font-medium text-foreground">Document Sync</label>
+                          <label className="block text-sm font-medium text-foreground">Server Document Sync</label>
                           <div className="flex gap-2">
                             <Button
                               onClick={handleLoad}
@@ -610,7 +610,7 @@ export function SettingsModal() {
                                        transform transition-transform duration-200 ease-in-out hover:scale-[1.04] hover:text-accent
                                        disabled:opacity-50"
                             >
-                              {isLoading ? `Loading... ${Math.round(progress)}%` : 'Load docs from Server'}
+                              {isLoading ? `Loading... ${Math.round(progress)}%` : 'Load'}
                             </Button>
                             <Button
                               onClick={handleSync}
@@ -621,13 +621,13 @@ export function SettingsModal() {
                                        transform transition-transform duration-200 ease-in-out hover:scale-[1.04] hover:text-accent
                                        disabled:opacity-50"
                             >
-                              {isSyncing ? `Saving... ${Math.round(progress)}%` : 'Save local to Server'}
+                              {isSyncing ? `Saving... ${Math.round(progress)}%` : 'Save to server'}
                             </Button>
                           </div>
                         </div>}
 
                         <div className="space-y-1 pb-3">
-                          <label className="block text-sm font-medium text-foreground">Bulk Delete</label>
+                          <label className="block text-sm font-medium text-foreground">Delete All</label>
                           <div className="flex gap-2">
                             <Button
                               onClick={() => setShowClearLocalConfirm(true)}
@@ -636,7 +636,7 @@ export function SettingsModal() {
                                          focus-visible:ring-2 focus-visible:bg-red-500 focus-visible:ring-offset-2
                                        transform transition-transform duration-200 ease-in-out hover:scale-[1.04]"
                             >
-                              Delete local docs
+                              Delete local
                             </Button>
                             {isDev && <Button
                               onClick={() => setShowClearServerConfirm(true)}
@@ -645,7 +645,7 @@ export function SettingsModal() {
                                          focus-visible:ring-2 focus-visible:bg-red-500 focus-visible:ring-offset-2
                                        transform transition-transform duration-200 ease-in-out hover:scale-[1.04]"
                             >
-                              Delete server docs
+                              Delete server
                             </Button>}
                           </div>
                         </div>
diff --git a/tests/helpers.ts b/tests/helpers.ts
index 6b1557c..f200a7a 100644
--- a/tests/helpers.ts
+++ b/tests/helpers.ts
@@ -199,7 +199,7 @@ export async function openSettingsDocumentsTab(page: Page) {
 // Delete all local documents through Settings and close dialogs
 export async function deleteAllLocalDocuments(page: Page) {
   await openSettingsDocumentsTab(page);
-  await page.getByRole('button', { name: 'Delete local docs' }).click();
+  await page.getByRole('button', { name: 'Delete local' }).click();
 
   const heading = page.getByRole('heading', { name: 'Delete Local Documents' });
   await expect(heading).toBeVisible({ timeout: 10000 });