From 320834b60d896657e4787b006388676a0e01e3ab Mon Sep 17 00:00:00 2001
From: hkonsti <45428767+hkonsti@users.noreply.github.com>
Date: Wed, 11 Nov 2026 18:22:13 +0000
Subject: [PATCH] make the notifications setting a choice between on and off,
 as the theme is

---
 .../src/components/notification-setting.tsx   | 21 ++++++++++++-------
 1 file changed, 13 insertions(+), 8 deletions(-)

diff --git a/packages/web/src/components/notification-setting.tsx b/packages/web/src/components/notification-setting.tsx
index 4a23c677..cc0db15d 100644
--- a/packages/web/src/components/notification-setting.tsx
+++ b/packages/web/src/components/notification-setting.tsx
@@ -1,4 +1,5 @@
 import {useState} from 'react';
+import {BellIcon, BellOffIcon} from 'lucide-react';
 import {PillButton} from '@proxy/ui/components/pill-button';
 import {
 	notificationsBlocked,
@@ -7,9 +8,9 @@ import {
 	turnNotificationsOn,
 } from '@/lib/notifications';
 
-// Notifications of what the agents say while the person is elsewhere, on this device: on or off,
-// with the button that turns them the other way; where the browser has refused the page its
-// leave, the way to change that is the browser's own.
+// Notifications of what the agents say while the person is elsewhere, on this device: one button
+// for on and one for off, the one in use pressed, as the theme picker looks; where the browser has
+// refused the page its leave, the way to change that is the browser's own.
 export function NotificationSetting() {
 	const [on, setOn] = useState(notificationsOn);
 	const [blocked, setBlocked] = useState(notificationsBlocked);
@@ -30,11 +31,15 @@ export function NotificationSetting() {
 	}
 
 	return (
-		<span className="flex items-center gap-3">
-			<span>{on ? 'On' : 'Off'}</span>
-			<PillButton variant="ghost" onClick={on ? turnOff : () => void turnOn()}>
-				{on ? 'Turn off' : 'Turn on'}
+		<div role="group" aria-label="Notifications" className="flex items-center gap-2">
+			<PillButton variant="ghost" aria-pressed={on} onClick={() => void turnOn()}>
+				<BellIcon data-icon="inline-start" />
+				On
 			</PillButton>
-		</span>
+			<PillButton variant="ghost" aria-pressed={!on} onClick={turnOff}>
+				<BellOffIcon data-icon="inline-start" />
+				Off
+			</PillButton>
+		</div>
 	);
 }