From ecbcd5803b566eb7d3046a088724951eabfe7863 Mon Sep 17 00:00:00 2001
From: yyh <yuanyouhuilyz@gmail.com>
Date: Fri, 6 Feb 2026 13:49:08 +0800
Subject: [PATCH] fix(workflow): avoid nested button in skill file tree menu

---
 .../components/workflow/skill/file-tree/menu-item.tsx  | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/web/app/components/workflow/skill/file-tree/menu-item.tsx b/web/app/components/workflow/skill/file-tree/menu-item.tsx
index d76b1cf391..7d681e3bae 100644
--- a/web/app/components/workflow/skill/file-tree/menu-item.tsx
+++ b/web/app/components/workflow/skill/file-tree/menu-item.tsx
@@ -81,13 +81,15 @@ const MenuItem = ({ icon: Icon, label, kbd, onClick, disabled, variant, tooltip
           popupContent={tooltip}
           position="right"
         >
-          <button
-            type="button"
+          <span
             className="flex shrink-0 items-center justify-center"
-            onClick={e => e.stopPropagation()}
+            onClick={(event) => {
+              event.preventDefault()
+              event.stopPropagation()
+            }}
           >
             <RiQuestionLine className="size-4 text-text-quaternary hover:text-text-tertiary" />
-          </button>
+          </span>
         </Tooltip>
       )}
     </button>