From a48e994dbf9ee481ae23cabc428a1a6a7cdf4fcf Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Tue, 14 Apr 2026 20:21:58 +0000
Subject: [PATCH] Add descriptive labels to STOP button

Co-authored-by: manupawickramasinghe <73810867+manupawickramasinghe@users.noreply.github.com>
---
 .Jules/palette.md                           | 4 ++++
 app/src/main/res/layout/fragment_camera.xml | 4 +++-
 app/src/main/res/values/strings.xml         | 1 +
 3 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/.Jules/palette.md b/.Jules/palette.md
index 5e5130d..ae48fec 100644
--- a/.Jules/palette.md
+++ b/.Jules/palette.md
@@ -29,3 +29,7 @@
 ## 2026-04-11 - Prevent unreadable hardcoded text sizing
 **Learning:** Hardcoding `android:textSize="12sp"` directly on Android TextView elements creates an accessibility barrier for users with poor vision and ignores system-level font scaling preferences, particularly for dynamic data fields like bitrate readouts.
 **Action:** Remove overly restrictive `android:textSize` attributes from informational UI elements, allowing them to inherit accessible default text sizes from the application theme.
+
+## 2026-04-14 - Descriptive Labels for Brief/Destructive Actions
+**Learning:** Brief or destructive action buttons (like 'STOP') may lack context for screen reader users and sighted users hovering with a mouse, especially when the visual label is short.
+**Action:** Enhance brief or destructive buttons with `android:contentDescription` and `android:tooltipText` that explicitly describe their full outcome (e.g., 'Stop camera server') to provide clear context across input modalities.
diff --git a/app/src/main/res/layout/fragment_camera.xml b/app/src/main/res/layout/fragment_camera.xml
index 3714d5d..2184e5a 100644
--- a/app/src/main/res/layout/fragment_camera.xml
+++ b/app/src/main/res/layout/fragment_camera.xml
@@ -90,7 +90,9 @@
                             android:layout_height="wrap_content"
                             android:layout_gravity="end|fill_vertical"
                             android:includeFontPadding="false"
-                            android:text="@string/stop_button" />
+                            android:text="@string/stop_button"
+                            android:contentDescription="@string/stop_button_desc"
+                            android:tooltipText="@string/stop_button_desc" />
                     </LinearLayout>
 
                 </TableRow>
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index f77ec82..39136d3 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -20,6 +20,7 @@
     <!-- Camera Fragment Strings -->
     <string name="app_title">RemoteCam</string>
     <string name="stop_button">STOP</string>
+    <string name="stop_button_desc">Stop camera server</string>
     <string name="preview_label">Preview</string>
     <string name="stream_label">Stream</string>
     <string name="rtsp_label">RTSP</string>