From 4c6610d62bfb81b7f7d801aeb8c9307420f178a5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E5=91=A8=E5=87=AF?= <851708184@qq.com>
Date: Fri, 23 Oct 2026 16:59:29 +0800
Subject: [PATCH] fix(web): cap dialog height so long content keeps actions
 reachable

Dialog content is fixed-centered with no height limit, so a long skill
body (the textarea uses field-sizing-content) grows the dialog past the
viewport and the footer actions end up unreachable because the page
itself does not scroll.

Cap the shared dialog at the viewport height with internal scrolling.
This also fixes other long-form dialogs (e.g. the memory editor) in one
place.

Closes #429
---
 web/src/lib/components/ui/dialog.svelte | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web/src/lib/components/ui/dialog.svelte b/web/src/lib/components/ui/dialog.svelte
index 2d690100..232061a0 100644
--- a/web/src/lib/components/ui/dialog.svelte
+++ b/web/src/lib/components/ui/dialog.svelte
@@ -34,7 +34,7 @@
     />
     <DialogPrimitive.Content
       class={cn(
-        "bg-popover text-popover-foreground data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0 data-[state=closed]:zoom-out-95 data-[state=open]:zoom-in-95 ring-foreground/5 dark:ring-foreground/10 grid max-w-[calc(100%-2rem)] gap-6 rounded-[min(var(--radius-4xl),24px)] p-6 text-sm shadow-xl ring-1 duration-100 sm:max-w-md fixed top-1/2 left-1/2 z-50 w-full -translate-x-1/2 -translate-y-1/2 outline-none [--floating-label-bg:var(--popover)]",
+        "bg-popover text-popover-foreground data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0 data-[state=closed]:zoom-out-95 data-[state=open]:zoom-in-95 ring-foreground/5 dark:ring-foreground/10 grid max-h-[calc(100dvh-2rem)] max-w-[calc(100%-2rem)] gap-6 overflow-y-auto rounded-[min(var(--radius-4xl),24px)] p-6 text-sm shadow-xl ring-1 duration-100 sm:max-w-md fixed top-1/2 left-1/2 z-50 w-full -translate-x-1/2 -translate-y-1/2 outline-none [--floating-label-bg:var(--popover)]",
         className
       )}
     >