diff --git a/web/src/app/(main)/system/agents/page.tsx b/web/src/app/(main)/system/agents/page.tsx index 7add813..683c6ca 100644 --- a/web/src/app/(main)/system/agents/page.tsx +++ b/web/src/app/(main)/system/agents/page.tsx @@ -258,15 +258,6 @@ export default function AgentsPage() { { - const t = e.detail.originalEvent.target as Element | null; - if (t?.closest?.("[data-radix-popper-content-wrapper],[data-radix-select-content],[role='listbox']")) { - e.preventDefault(); - } - }} > {editing && ( <> diff --git a/web/src/components/ui/dialog.tsx b/web/src/components/ui/dialog.tsx index 230ffd5..af8e78d 100644 --- a/web/src/components/ui/dialog.tsx +++ b/web/src/components/ui/dialog.tsx @@ -3,7 +3,7 @@ import * as React from "react" import { Dialog as DialogPrimitive } from "radix-ui" -import { cn } from "@/lib/utils" +import { cn, radixOverlayWasOpenAtPointerDown } from "@/lib/utils" import { Button } from "@/components/ui/button" import { XIcon } from "lucide-react" @@ -51,6 +51,7 @@ function DialogContent({ className, children, showCloseButton = true, + onInteractOutside, ...props }: React.ComponentProps & { showCloseButton?: boolean @@ -60,6 +61,18 @@ function DialogContent({ { + const target = e.detail.originalEvent.target as Element | null + const onOverlay = !!target?.closest?.("[data-slot='dialog-overlay']") + if (!onOverlay || radixOverlayWasOpenAtPointerDown()) e.preventDefault() + onInteractOutside?.(e) + }} className={cn( "fixed top-1/2 left-1/2 z-50 grid max-h-[90dvh] w-full max-w-[calc(100%-2rem)] -translate-x-1/2 -translate-y-1/2 gap-4 overflow-y-auto rounded-xl bg-popover p-4 text-sm text-popover-foreground ring-1 ring-foreground/10 duration-100 outline-none sm:max-w-sm data-open:animate-in data-open:fade-in-0 data-open:zoom-in-95 data-closed:animate-out data-closed:fade-out-0 data-closed:zoom-out-95", className diff --git a/web/src/components/ui/sheet.tsx b/web/src/components/ui/sheet.tsx index 8d0e1d3..ce2b56a 100644 --- a/web/src/components/ui/sheet.tsx +++ b/web/src/components/ui/sheet.tsx @@ -3,7 +3,7 @@ import * as React from "react" import { Dialog as SheetPrimitive } from "radix-ui" -import { cn } from "@/lib/utils" +import { cn, radixOverlayWasOpenAtPointerDown } from "@/lib/utils" import { Button } from "@/components/ui/button" import { XIcon } from "lucide-react" @@ -50,6 +50,7 @@ function SheetContent({ children, side = "right", showCloseButton = true, + onInteractOutside, ...props }: React.ComponentProps & { side?: "top" | "right" | "bottom" | "left" @@ -61,6 +62,18 @@ function SheetContent({ { + const target = e.detail.originalEvent.target as Element | null + const onOverlay = !!target?.closest?.("[data-slot='sheet-overlay']") + if (!onOverlay || radixOverlayWasOpenAtPointerDown()) e.preventDefault() + onInteractOutside?.(e) + }} className={cn( "fixed z-50 flex flex-col gap-4 bg-popover bg-clip-padding text-sm text-popover-foreground shadow-lg transition duration-200 ease-in-out data-[side=bottom]:inset-x-0 data-[side=bottom]:bottom-0 data-[side=bottom]:h-auto data-[side=bottom]:border-t data-[side=left]:inset-y-0 data-[side=left]:left-0 data-[side=left]:h-full data-[side=left]:w-3/4 data-[side=left]:border-r data-[side=right]:inset-y-0 data-[side=right]:right-0 data-[side=right]:h-full data-[side=right]:w-3/4 data-[side=right]:border-l data-[side=top]:inset-x-0 data-[side=top]:top-0 data-[side=top]:h-auto data-[side=top]:border-b data-[side=left]:sm:max-w-sm data-[side=right]:sm:max-w-sm data-open:animate-in data-open:fade-in-0 data-[side=bottom]:data-open:slide-in-from-bottom-10 data-[side=left]:data-open:slide-in-from-left-10 data-[side=right]:data-open:slide-in-from-right-10 data-[side=top]:data-open:slide-in-from-top-10 data-closed:animate-out data-closed:fade-out-0 data-[side=bottom]:data-closed:slide-out-to-bottom-10 data-[side=left]:data-closed:slide-out-to-left-10 data-[side=right]:data-closed:slide-out-to-right-10 data-[side=top]:data-closed:slide-out-to-top-10", className diff --git a/web/src/lib/utils.ts b/web/src/lib/utils.ts index dda1f92..eb6d2c9 100644 --- a/web/src/lib/utils.ts +++ b/web/src/lib/utils.ts @@ -5,6 +5,46 @@ export function cn(...inputs: ClassValue[]) { return twMerge(clsx(inputs)); } +// 抽屉/对话框(Sheet/Dialog)的 onInteractOutside 关闭判定辅助。 +// +// 背景:抽屉内的 Radix 弹层(Select 下拉、DropdownMenu、Popover 等)会 portal 到抽屉 +// 之外。开着弹层时点遮罩/抽屉外想收起它,这一次 pointerdown 会被 Select 和 Sheet 两个 +// DismissableLayer 同时处理;Select 先关闭且是 discrete 事件、React 会同步 flush,于是 +// 轮到 Sheet 的处理器时弹层的 data-state 早已翻成 closed —— 在"当下"检测弹层是否打开 +// 天然不可靠(实测已验证)。 +// +// 正确做法:Radix 的 pointerdown 监听在冒泡阶段;我们在 capture 阶段(早于它)先把 +// "此刻有没有弹层开着"记录下来,onInteractOutside 再读这个记录值来决定是否放行关闭。 +function isRadixOverlayOpenNow(): boolean { + if (typeof document === "undefined") return false; + return !!document.querySelector( + [ + "[data-slot='select-trigger'][data-state='open']", + "[data-slot='select-content'][data-state='open']", + "[role='listbox'][data-state='open']", + "[data-radix-popper-content-wrapper]", + "[aria-expanded='true'][data-state='open']", + ].join(","), + ); +} + +let overlayOpenAtLastPointerDown = false; +if (typeof document !== "undefined") { + document.addEventListener( + "pointerdown", + () => { + overlayOpenAtLastPointerDown = isRadixOverlayOpenNow(); + }, + true, // capture:抢在 Radix 冒泡阶段的 pointerdown 处理器之前记录 + ); +} + +// radixOverlayWasOpenAtPointerDown 返回"最近一次 pointerdown 发生时是否有 Radix 弹层 +// 开着"。抽屉/对话框据此:开着弹层时点遮罩 → 只收弹层、不关自身。 +export function radixOverlayWasOpenAtPointerDown(): boolean { + return overlayOpenAtLastPointerDown; +} + export const getInitials = (str: string): string => { if (typeof str !== "string" || !str.trim()) return "?";