From 940935f563ba53a613eac713913bc818d19bc11c Mon Sep 17 00:00:00 2001
From: Taylor Ho <taylorkmho@gmail.com>
Date: Wed, 20 May 2026 08:58:40 -0700
Subject: [PATCH] fix: clip QuickAddAgentPopover items to container
 border-radius

Add overflow-hidden to PopoverContent so hover backgrounds on the
bottom item stay clipped within the rounded corners. Also restore
role=menu on the keyboard-navigable container (lost during revert).
---
 .../features/channels/ui/QuickAddAgentPopover.tsx   | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/desktop/src/features/channels/ui/QuickAddAgentPopover.tsx b/desktop/src/features/channels/ui/QuickAddAgentPopover.tsx
index b3ffbe4a7b..5cda817d70 100644
--- a/desktop/src/features/channels/ui/QuickAddAgentPopover.tsx
+++ b/desktop/src/features/channels/ui/QuickAddAgentPopover.tsx
@@ -301,9 +301,14 @@ export function QuickAddAgentPopover({
   return (
     <Popover open={open} onOpenChange={onOpenChange}>
       <PopoverTrigger asChild>{children}</PopoverTrigger>
-      <PopoverContent align="end" className="w-72 p-0" sideOffset={6}>
+      <PopoverContent
+        align="end"
+        className="w-72 overflow-hidden p-0"
+        sideOffset={6}
+      >
         <div
           className="flex max-h-80 flex-col"
+          role="menu"
           onKeyDown={(e) => {
             const container = e.currentTarget;
             const buttons = Array.from(
@@ -344,7 +349,11 @@ export function QuickAddAgentPopover({
                 No agents available.
               </div>
             ) : (
-              <div aria-label="Available agents" className="py-1" role="listbox">
+              <div
+                aria-label="Available agents"
+                className="py-1"
+                role="listbox"
+              >
                 {items.map((item) => {
                   const itemKey = getItemKey(item);
                   const isInChannel = item.kind === "running-in-channel";