From 4706e0666bd26869e1b9a094faa993917533cbde Mon Sep 17 00:00:00 2001
From: mlogclub <mlog1@qq.com>
Date: Tue, 2 Jun 2026 19:57:09 +0800
Subject: [PATCH] feat: add click handling for detail dialog in DocumentList
 and FAQList components

---
 web/app/dashboard/knowledge/_components/document-list.tsx | 8 ++++++--
 web/app/dashboard/knowledge/_components/faq-list.tsx      | 4 +++-
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/web/app/dashboard/knowledge/_components/document-list.tsx b/web/app/dashboard/knowledge/_components/document-list.tsx
index e2824d0..a61ee2f 100644
--- a/web/app/dashboard/knowledge/_components/document-list.tsx
+++ b/web/app/dashboard/knowledge/_components/document-list.tsx
@@ -509,9 +509,10 @@ export function DocumentList({ knowledgeBaseId, onActionStateChange }: DocumentL
                   <ContextMenuTrigger className="w-full">
                     <div
                       className={cn(
-                        "bg-background p-3 transition-colors hover:bg-accent w-full",
+                        "bg-background p-3 transition-colors hover:bg-accent w-full cursor-pointer",
                         contextMenuDocumentId === item.id && "bg-accent text-accent-foreground",
                       )}
+                      onClick={() => openDetailDialog(item)}
                     >
                       <div className="flex items-start justify-between gap-2">
                         <div className="flex min-w-0 flex-1 items-start gap-2">
@@ -541,6 +542,7 @@ export function DocumentList({ knowledgeBaseId, onActionStateChange }: DocumentL
                         </div>
                         <DropdownMenu>
                           <DropdownMenuTrigger
+                            onClick={(event) => event.stopPropagation()}
                             render={
                               <Button
                                 variant="ghost"
@@ -616,9 +618,10 @@ export function DocumentList({ knowledgeBaseId, onActionStateChange }: DocumentL
                   <ContextMenuTrigger className="w-full">
                     <div
                       className={cn(
-                        "flex items-center gap-3 bg-background p-2 transition-colors hover:bg-accent w-full",
+                        "flex items-center gap-3 bg-background p-2 transition-colors hover:bg-accent w-full cursor-pointer",
                         contextMenuDocumentId === item.id && "bg-accent text-accent-foreground",
                       )}
+                      onClick={() => openDetailDialog(item)}
                     >
                       <Checkbox
                         checked={selectedIds.includes(item.id)}
@@ -643,6 +646,7 @@ export function DocumentList({ knowledgeBaseId, onActionStateChange }: DocumentL
                       </div>
                       <DropdownMenu>
                         <DropdownMenuTrigger
+                          onClick={(event) => event.stopPropagation()}
                           render={
                             <Button
                               variant="ghost"
diff --git a/web/app/dashboard/knowledge/_components/faq-list.tsx b/web/app/dashboard/knowledge/_components/faq-list.tsx
index db377b8..c19176f 100644
--- a/web/app/dashboard/knowledge/_components/faq-list.tsx
+++ b/web/app/dashboard/knowledge/_components/faq-list.tsx
@@ -454,9 +454,10 @@ export function FAQList({
                     <ContextMenuTrigger className="w-full">
                       <div
                         className={cn(
-                          "flex items-center gap-3 bg-background p-2 transition-colors hover:bg-accent w-full",
+                          "flex items-center gap-3 bg-background p-2 transition-colors hover:bg-accent w-full cursor-pointer",
                           contextMenuFAQId === item.id && "bg-accent text-accent-foreground",
                         )}
+                        onClick={() => openDetailDialog(item)}
                       >
                         <Checkbox
                           checked={selectedIds.includes(item.id)}
@@ -478,6 +479,7 @@ export function FAQList({
                         </div>
                         <DropdownMenu>
                           <DropdownMenuTrigger
+                            onClick={(event) => event.stopPropagation()}
                             render={<Button variant="ghost" size="icon" className="size-6" />}
                             aria-label={t("knowledge.moreActions", { name: item.question })}
                           >