From c01430e53b3370f21aaf1db3b444b1f2cde7ab26 Mon Sep 17 00:00:00 2001
From: Danielle Maywood <danielle@themaywoods.com>
Date: Sun, 8 Mar 2026 17:45:18 +0000
Subject: [PATCH] fix(site): add missing aria-labels to AgentsPage icon buttons
 (#22817)

---
 .../pages/AgentsPage/AgentDetail/ConversationTimeline.tsx    | 2 ++
 .../pages/AgentsPage/ChatModelAdminPanel/ModelsSection.tsx   | 5 +++++
 site/src/pages/AgentsPage/ChimeButton.tsx                    | 3 +++
 site/src/pages/AgentsPage/DiffStats.tsx                      | 1 +
 site/src/pages/AgentsPage/WebPushButton.tsx                  | 5 +++++
 5 files changed, 16 insertions(+)

diff --git a/site/src/pages/AgentsPage/AgentDetail/ConversationTimeline.tsx b/site/src/pages/AgentsPage/AgentDetail/ConversationTimeline.tsx
index 5867d2328f..127b33ba2e 100644
--- a/site/src/pages/AgentsPage/AgentDetail/ConversationTimeline.tsx
+++ b/site/src/pages/AgentsPage/AgentDetail/ConversationTimeline.tsx
@@ -207,6 +207,7 @@ function renderBlockList({
 							<button
 								key={`${keyPrefix}-file-${index}`}
 								type="button"
+								aria-label="View image"
 								className="inline-block rounded-md border-0 bg-transparent p-0"
 								onClick={(e) => {
 									e.stopPropagation();
@@ -347,6 +348,7 @@ const ChatMessageItem = memo<{
 													<button
 														key={`user-file-${i}`}
 														type="button"
+														aria-label="View image"
 														className="inline-block rounded-md border-0 bg-transparent p-0"
 														onClick={(e) => {
 															e.stopPropagation();
diff --git a/site/src/pages/AgentsPage/ChatModelAdminPanel/ModelsSection.tsx b/site/src/pages/AgentsPage/ChatModelAdminPanel/ModelsSection.tsx
index ffbb1c7c2b..a71e30c8af 100644
--- a/site/src/pages/AgentsPage/ChatModelAdminPanel/ModelsSection.tsx
+++ b/site/src/pages/AgentsPage/ChatModelAdminPanel/ModelsSection.tsx
@@ -199,6 +199,11 @@ export const ModelsSection: FC<ModelsSectionProps> = ({
 											handleSetDefault(modelConfig);
 										}}
 										aria-disabled={isUpdating || modelConfig.is_default}
+										aria-label={
+											modelConfig.is_default
+												? "Default model"
+												: "Set as default model"
+										}
 										className={cn(
 											"flex h-7 w-7 shrink-0 items-center justify-center rounded-md bg-transparent border-0 p-0 transition-colors",
 											modelConfig.is_default
diff --git a/site/src/pages/AgentsPage/ChimeButton.tsx b/site/src/pages/AgentsPage/ChimeButton.tsx
index 982d18fca1..367618c2c7 100644
--- a/site/src/pages/AgentsPage/ChimeButton.tsx
+++ b/site/src/pages/AgentsPage/ChimeButton.tsx
@@ -24,6 +24,9 @@ export const ChimeButton: FC = () => {
 					variant="subtle"
 					size="icon"
 					onClick={handleClick}
+					aria-label={
+						enabled ? "Mute completion chime" : "Enable completion chime"
+					}
 					className="h-7 w-7 text-content-secondary hover:text-content-primary"
 				>
 					{enabled ? (
diff --git a/site/src/pages/AgentsPage/DiffStats.tsx b/site/src/pages/AgentsPage/DiffStats.tsx
index 4359d6efe8..c48b9e39c3 100644
--- a/site/src/pages/AgentsPage/DiffStats.tsx
+++ b/site/src/pages/AgentsPage/DiffStats.tsx
@@ -78,6 +78,7 @@ export const DiffStatsInline: FC<{
 		<button
 			type="button"
 			onClick={onClick}
+			aria-label="View diff statistics"
 			className="inline-flex shrink-0 cursor-pointer items-center border-0 bg-transparent p-0 leading-none transition-opacity hover:opacity-80 outline-none"
 		>
 			<DiffStatNumbers additions={additions} deletions={deletions} />
diff --git a/site/src/pages/AgentsPage/WebPushButton.tsx b/site/src/pages/AgentsPage/WebPushButton.tsx
index 35af62e4a1..e425ae2c51 100644
--- a/site/src/pages/AgentsPage/WebPushButton.tsx
+++ b/site/src/pages/AgentsPage/WebPushButton.tsx
@@ -43,6 +43,11 @@ export const WebPushButton: FC = () => {
 					size="icon"
 					disabled={webPush.loading}
 					onClick={handleClick}
+					aria-label={
+						webPush.subscribed
+							? "Disable notifications"
+							: "Enable notifications"
+					}
 					className="h-7 w-7 text-content-secondary hover:text-content-primary"
 				>
 					{webPush.loading ? (