From b1f7840e45a8256fd9843e5fb79d6bca397cd019 Mon Sep 17 00:00:00 2001
From: Jason <farion1231@gmail.com>
Date: Sat, 1 Nov 2025 18:59:19 +0800
Subject: [PATCH] fix(provider): add footer and improve layout spacing for
 common config editor dialog

- Add DialogFooter with Cancel and Save buttons to match Codex common config modal
- Improve dialog layout spacing with proper padding (px-6, py-4)
- Add flex layout with overflow handling for better responsiveness
- Fix content being too close to dialog edges
---
 .../providers/forms/CommonConfigEditor.tsx     | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/src/components/providers/forms/CommonConfigEditor.tsx b/src/components/providers/forms/CommonConfigEditor.tsx
index 8415989..1afa84f 100644
--- a/src/components/providers/forms/CommonConfigEditor.tsx
+++ b/src/components/providers/forms/CommonConfigEditor.tsx
@@ -4,8 +4,11 @@ import {
   DialogContent,
   DialogHeader,
   DialogTitle,
+  DialogFooter,
 } from "@/components/ui/dialog";
 import { Label } from "@/components/ui/label";
+import { Button } from "@/components/ui/button";
+import { Save } from "lucide-react";
 
 interface CommonConfigEditorProps {
   value: string;
@@ -105,15 +108,15 @@ export function CommonConfigEditor({
         open={isModalOpen}
         onOpenChange={(open) => !open && onModalClose()}
       >
-        <DialogContent className="sm:max-w-[600px]">
-          <DialogHeader>
+        <DialogContent className="sm:max-w-[600px] max-h-[90vh] flex flex-col p-0">
+          <DialogHeader className="px-6 pt-6 pb-0">
             <DialogTitle>
               {t("claudeConfig.editCommonConfigTitle", {
                 defaultValue: "编辑通用配置片段",
               })}
             </DialogTitle>
           </DialogHeader>
-          <div className="space-y-4 py-4">
+          <div className="flex-1 overflow-auto px-6 py-4 space-y-4">
             <p className="text-sm text-muted-foreground">
               {t("claudeConfig.commonConfigHint", {
                 defaultValue: "通用配置片段将合并到所有启用它的供应商配置中",
@@ -140,6 +143,15 @@ export function CommonConfigEditor({
               </p>
             )}
           </div>
+          <DialogFooter>
+            <Button type="button" variant="outline" onClick={onModalClose}>
+              {t("common.cancel")}
+            </Button>
+            <Button type="button" onClick={onModalClose} className="gap-2">
+              <Save className="w-4 h-4" />
+              {t("common.save")}
+            </Button>
+          </DialogFooter>
         </DialogContent>
       </Dialog>
     </>