From b3c00fe6da5fba4293bb4ea87b60e056a3dc24fc Mon Sep 17 00:00:00 2001
From: RockChinQ <rockchinq@gmail.com>
Date: Tue, 16 Jun 2026 05:18:14 -0400
Subject: [PATCH] fix(web): use fixed height for settings dialog instead of
 80vh

Avoid the dialog stretching to fill tall viewports (large empty space).
Pin to 620px with max-h-[85vh] fallback and narrow width to 52rem.
---
 .../app/home/components/settings-dialog/SettingsDialog.tsx  | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/web/src/app/home/components/settings-dialog/SettingsDialog.tsx b/web/src/app/home/components/settings-dialog/SettingsDialog.tsx
index f22fc779f..9b24b9e21 100644
--- a/web/src/app/home/components/settings-dialog/SettingsDialog.tsx
+++ b/web/src/app/home/components/settings-dialog/SettingsDialog.tsx
@@ -116,7 +116,7 @@ export default function SettingsDialog({
       }}
     >
       <DialogContent
-        className="overflow-hidden p-0 sm:max-w-[56rem] [&>button:last-child]:z-20"
+        className="overflow-hidden p-0 sm:max-w-[52rem] [&>button:last-child]:z-20"
         // The dialog itself is the scroll boundary; each panel manages its own
         // internal scrolling.
       >
@@ -128,7 +128,7 @@ export default function SettingsDialog({
         <SidebarProvider className="items-start">
           <Sidebar
             collapsible="none"
-            className="hidden h-[80vh] w-56 shrink-0 border-r md:flex"
+            className="hidden h-[620px] max-h-[85vh] w-56 shrink-0 border-r md:flex"
           >
             <SidebarContent>
               <SidebarGroup>
@@ -154,7 +154,7 @@ export default function SettingsDialog({
             </SidebarContent>
           </Sidebar>
 
-          <main className="flex h-[80vh] min-w-0 flex-1 flex-col overflow-hidden">
+          <main className="flex h-[620px] max-h-[85vh] min-w-0 flex-1 flex-col overflow-hidden">
             {/* Mobile section switcher (sidebar is hidden on small screens) */}
             <div className="flex shrink-0 items-center gap-1 overflow-x-auto border-b px-3 py-2 md:hidden">
               {navItems.map((item) => (