From 2b63c20180db712951f100739e6cfca9d29e2a99 Mon Sep 17 00:00:00 2001
From: Matt Toohey <contact@matttoohey.com>
Date: Fri, 8 May 2026 15:28:50 +1000
Subject: [PATCH] fix(desktop): prevent Radix Checkbox from swallowing persona
 row clicks
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Radix UI's Checkbox internally calls stopPropagation on click events,
which prevents the parent row's onClick from firing. This made clicking
near the checkbox area unreliable — either nothing happened or a
double-toggle cancelled out.

Make the Checkbox purely presentational by removing onCheckedChange,
adding pointer-events-none so clicks pass through to the row handler,
and tabIndex={-1} to avoid double-focus since the row is already
focusable.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Matt Toohey <contact@matttoohey.com>
---
 desktop/src/features/agents/ui/TeamDialog.tsx | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/desktop/src/features/agents/ui/TeamDialog.tsx b/desktop/src/features/agents/ui/TeamDialog.tsx
index 9636bd9e7..2a4259aec 100644
--- a/desktop/src/features/agents/ui/TeamDialog.tsx
+++ b/desktop/src/features/agents/ui/TeamDialog.tsx
@@ -416,8 +416,9 @@ export function TeamDialog({
                         >
                           <Checkbox
                             checked={isSelected}
+                            className="pointer-events-none"
                             disabled={isPending}
-                            onCheckedChange={() => togglePersona(persona.id)}
+                            tabIndex={-1}
                           />
                           <ProfileAvatar
                             avatarUrl={persona.avatarUrl}