debugging

Author: liuliu-dev

packages/react/src/ActionList/Item.tsx

@@ -71,9 +71,19 @@ const LinkItemContainerNoBox = React.forwardRef<HTMLAnchorElement, LinkItemConta
 
       // Check if click is on a button/svg/path (menu trigger) and prevent navigation immediately
       const targetTag = (event.target as HTMLElement)?.tagName?.toLowerCase()
-      if (['button', 'svg', 'path'].includes(targetTag)) {
-        console.log('[LinkItemContainerNoBox] Click on button/svg/path detected, preventing default')
+      const isMenuTrigger = ['button', 'svg', 'path'].includes(targetTag)
+      console.log('[LinkItemContainerNoBox] Checking target', {
+        targetTag,
+        targetTagRaw: (event.target as HTMLElement)?.tagName,
+        isButton: targetTag === 'button',
+        isSvg: targetTag === 'svg',
+        isPath: targetTag === 'path',
+        willPreventDefault: isMenuTrigger,
+      })
+      if (isMenuTrigger) {
+        console.log('[LinkItemContainerNoBox] Click on button/svg/path detected, preventing default AND stopping propagation')
         event.preventDefault()
+        event.stopPropagation()
       }
 
       // Call Item's onClick first (handles selection logic)
@@ -102,9 +112,15 @@ const LinkItemContainerNoBox = React.forwardRef<HTMLAnchorElement, LinkItemConta
       hasOnClick: !!onClick,
       hasUserOnClick: !!userOnClick,
       restKeys: Object.keys(rest),
+      hasTo: 'to' in rest,
     })
+    
+    // Remove 'to' prop to prevent React Router from handling navigation
+    // The onClick handlers will manage navigation and menu opening
+    const {to: _, ...restWithoutTo} = rest as typeof rest & {to?: unknown}
+    
     return (
-      <Link as={Component} ref={forwardedRef} {...rest} onClick={clickHandler}>
+      <Link as={Component} ref={forwardedRef} {...restWithoutTo} onClick={clickHandler}>
         {children}
       </Link>
     )