From caaaaf571864aeb2bd68dc271972fd197c999bd1 Mon Sep 17 00:00:00 2001
From: gu yan <51285201+Slipstream-Max@users.noreply.github.com>
Date: Thu, 12 Feb 2026 20:18:01 +0800
Subject: [PATCH] fix: refactor layout containers for proxy and settings pages
 to ensure full height and unified scrolling (#102)

---
 src/routes/proxy.tsx    | 5 ++---
 src/routes/settings.tsx | 8 ++++----
 2 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/src/routes/proxy.tsx b/src/routes/proxy.tsx
index 9e0d0d8..a30c84d 100644
--- a/src/routes/proxy.tsx
+++ b/src/routes/proxy.tsx
@@ -192,21 +192,20 @@ print(response.choices[0].message.content)`;
 
   if (isLoading || !proxyConfig) {
     return (
-      <div className="flex h-screen items-center justify-center">
+      <div className="flex h-full items-center justify-center">
         <Loader2 className="animate-spin" />
       </div>
     );
   }
 
   return (
-    <div className="scrollbar-hide container mx-auto h-[calc(100vh-theme(spacing.16))] max-w-4xl space-y-6 overflow-y-auto p-6">
+    <div className="container mx-auto max-w-4xl space-y-5 p-6">
       <div>
         <h2 className="text-3xl font-bold tracking-tight">{t('proxy.title', 'API Proxy')}</h2>
         <p className="text-muted-foreground mt-1">
           {t('proxy.description', 'Manage the local API proxy service.')}
         </p>
 
-        {/* Local Access Info Banner */}
         {/* Local Access Info Banner */}
         {proxyConfig?.enabled && (
           <div className="mt-4 flex flex-col gap-2 rounded-md border border-blue-100 bg-blue-50 p-3 text-sm text-blue-900 dark:border-blue-900/50 dark:bg-blue-950/30 dark:text-blue-200">
diff --git a/src/routes/settings.tsx b/src/routes/settings.tsx
index d1846ac..5ed7dc5 100644
--- a/src/routes/settings.tsx
+++ b/src/routes/settings.tsx
@@ -66,14 +66,14 @@ function SettingsPage() {
 
   if (isLoading || !proxyConfig) {
     return (
-      <div className="flex h-screen items-center justify-center">
+      <div className="flex h-full items-center justify-center">
         <Loader2 className="animate-spin" />
       </div>
     );
   }
 
   return (
-    <div className="scrollbar-hide container mx-auto h-[calc(100vh-theme(spacing.16))] max-w-4xl space-y-8 overflow-y-auto p-6">
+    <div className="container mx-auto max-w-4xl space-y-5 p-6">
       <div>
         <h2 className="text-3xl font-bold tracking-tight">{t('settings.title')}</h2>
         <p className="text-muted-foreground mt-1">{t('settings.description')}</p>
@@ -86,7 +86,7 @@ function SettingsPage() {
         </TabsList>
 
         {/* --- GENERAL TAB --- */}
-        <TabsContent value="general" className="space-y-4">
+        <TabsContent value="general" className="space-y-5">
           <Card>
             <CardHeader>
               <CardTitle>{t('settings.appearance.title')}</CardTitle>
@@ -298,7 +298,7 @@ function SettingsPage() {
         </TabsContent>
 
         {/* --- PROXY TAB (Upstream Proxy Config Only) --- */}
-        <TabsContent value="proxy" className="space-y-4">
+        <TabsContent value="proxy" className="space-y-5">
           <Card>
             <CardHeader>
               <CardTitle>{t('settings.proxy.title')}</CardTitle>