From f0e43e9bcdcdd11283d0629ab04fc34e293e45ed Mon Sep 17 00:00:00 2001
From: kangfenmao <kangfenmao@qq.com>
Date: Sun, 18 May 2025 11:50:26 +0800
Subject: [PATCH] refactor: streamline SettingsTab layout and remove
 unnecessary Divider components

---
 src/renderer/src/pages/home/Tabs/SettingsTab.tsx | 11 +++++++----
 src/renderer/src/pages/settings/SettingGroup.tsx |  1 -
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/renderer/src/pages/home/Tabs/SettingsTab.tsx b/src/renderer/src/pages/home/Tabs/SettingsTab.tsx
index 2270566bbb..c7d4098bac 100644
--- a/src/renderer/src/pages/home/Tabs/SettingsTab.tsx
+++ b/src/renderer/src/pages/home/Tabs/SettingsTab.tsx
@@ -50,7 +50,7 @@ import {
   TranslateLanguageVarious
 } from '@renderer/types'
 import { modalConfirm } from '@renderer/utils'
-import { Button, Col, Divider, InputNumber, Row, Select, Slider, Switch, Tooltip } from 'antd'
+import { Button, Col, InputNumber, Row, Select, Slider, Switch, Tooltip } from 'antd'
 import { CircleHelp, RotateCcw, Settings2 } from 'lucide-react'
 import { FC, useCallback, useEffect, useMemo, useState } from 'react'
 import { useTranslation } from 'react-i18next'
@@ -236,7 +236,7 @@ const SettingsTab: FC<Props> = (props) => {
               />
             </Col>
           </Row>
-          <Divider style={{ margin: '10px 0' }} />
+          <SettingDivider />
           <SettingRow>
             <SettingRowTitleSmall>{t('models.stream_output')}</SettingRowTitleSmall>
             <Switch
@@ -249,7 +249,7 @@ const SettingsTab: FC<Props> = (props) => {
             />
           </SettingRow>
           <SettingDivider />
-          <Row align="middle" justify="space-between" style={{ marginBottom: 10 }}>
+          <SettingRow>
             <HStack alignItems="center">
               <Label>{t('chat.settings.max_tokens')}</Label>
               <Tooltip title={t('chat.settings.max_tokens.tip')}>
@@ -274,7 +274,7 @@ const SettingsTab: FC<Props> = (props) => {
                 onUpdateAssistantSettings({ enableMaxTokens: enabled })
               }}
             />
-          </Row>
+          </SettingRow>
           {enableMaxTokens && (
             <Row align="middle" gutter={10}>
               <Col span={24}>
@@ -292,6 +292,7 @@ const SettingsTab: FC<Props> = (props) => {
               </Col>
             </Row>
           )}
+          <SettingDivider />
         </SettingGroup>
       </CollapsibleSettingGroup>
       <CollapsibleSettingGroup title={t('settings.messages.title')} defaultExpanded={true}>
@@ -408,6 +409,7 @@ const SettingsTab: FC<Props> = (props) => {
               />
             </Col>
           </Row>
+          <SettingDivider />
         </SettingGroup>
       </CollapsibleSettingGroup>
       <CollapsibleSettingGroup title={t('chat.settings.code.title')} defaultExpanded={true}>
@@ -536,6 +538,7 @@ const SettingsTab: FC<Props> = (props) => {
             <Switch size="small" checked={codeWrappable} onChange={(checked) => dispatch(setCodeWrappable(checked))} />
           </SettingRow>
         </SettingGroup>
+        <SettingDivider />
       </CollapsibleSettingGroup>
       <CollapsibleSettingGroup title={t('settings.messages.input.title')} defaultExpanded={true}>
         <SettingGroup>
diff --git a/src/renderer/src/pages/settings/SettingGroup.tsx b/src/renderer/src/pages/settings/SettingGroup.tsx
index 4f0da0ce20..7948c20390 100644
--- a/src/renderer/src/pages/settings/SettingGroup.tsx
+++ b/src/renderer/src/pages/settings/SettingGroup.tsx
@@ -40,7 +40,6 @@ const SettingGroup = styled.div<{ theme?: ThemeMode }>`
   width: 100%;
   margin-top: 0;
   border-radius: 8px;
-  margin-bottom: 10px;
 `
 
 const GroupHeader = styled.div`