From 2462b17f64cf89e6facef4f78ca940bd7798c0ae Mon Sep 17 00:00:00 2001
From: Lucas Oliveira <62335616+lucaseduoli@users.noreply.github.com>
Date: Tue, 30 Sep 2025 17:49:42 -0300
Subject: [PATCH] fixed button inside of button, turning it into a div (#157)

---
 frontend/components/navigation.tsx | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/frontend/components/navigation.tsx b/frontend/components/navigation.tsx
index 339b7d22..c193b9f6 100644
--- a/frontend/components/navigation.tsx
+++ b/frontend/components/navigation.tsx
@@ -456,20 +456,24 @@ export function Navigation({
                             </div>
                           </div>
                           <DropdownMenu>
-                            <DropdownMenuTrigger asChild>
-                              <button
-                                type="button"
-                                className="opacity-0 group-hover:opacity-100 data-[state=open]:opacity-100 data-[state=open]:text-foreground transition-opacity p-1 hover:bg-accent rounded text-muted-foreground hover:text-foreground ml-2 flex-shrink-0"
+                            <DropdownMenuTrigger disabled={loading || deleteSessionMutation.isPending} asChild>
+                              <div
+                                className="opacity-0 group-hover:opacity-100 data-[state=open]:opacity-100 data-[state=open]:text-foreground transition-opacity p-1 hover:bg-accent rounded text-muted-foreground hover:text-foreground ml-2 flex-shrink-0 cursor-pointer"
                                 title="More options"
-                                disabled={
-                                  loading || deleteSessionMutation.isPending
-                                }
+                                role="button"
+                                tabIndex={0}
                                 onClick={(e) => {
                                   e.stopPropagation();
                                 }}
+                                onKeyDown={(e) => {
+                                  if (e.key === 'Enter' || e.key === ' ') {
+                                    e.preventDefault();
+                                    e.stopPropagation();
+                                  }
+                                }}
                               >
                                 <EllipsisVertical className="h-4 w-4" />
-                              </button>
+                              </div>
                             </DropdownMenuTrigger>
                             <DropdownMenuContent
                               side="bottom"