From 42f7df8f4ac097c521b21683dff607e39f4c3e4a Mon Sep 17 00:00:00 2001
From: jxxghp <jxxghp@gmail.com>
Date: Sat, 9 May 2026 21:40:35 +0800
Subject: [PATCH] fix: refine data cleanup settings tab

Move the data tab before log, hide retention fields until cleanup is enabled, and remove the extra download files prompt to keep the advanced settings flow focused.
---
 src/views/setting/AccountSettingSystem.vue | 145 ++++++++++-----------
 1 file changed, 71 insertions(+), 74 deletions(-)

diff --git a/src/views/setting/AccountSettingSystem.vue b/src/views/setting/AccountSettingSystem.vue
index 4fde21d0..7f067a33 100644
--- a/src/views/setting/AccountSettingSystem.vue
+++ b/src/views/setting/AccountSettingSystem.vue
@@ -1527,12 +1527,12 @@ watch(currentLlmSnapshotKey, (snapshotKey, previousSnapshotKey) => {
           <VTab value="network">
             <div>{{ t('setting.system.network') }}</div>
           </VTab>
-          <VTab value="log">
-            <div>{{ t('setting.system.log') }}</div>
-          </VTab>
           <VTab value="data">
             <div>{{ t('setting.system.data') }}</div>
           </VTab>
+          <VTab value="log">
+            <div>{{ t('setting.system.log') }}</div>
+          </VTab>
           <VTab value="dev">
             <div>{{ t('setting.system.lab') }}</div>
           </VTab>
@@ -1921,6 +1921,74 @@ watch(currentLlmSnapshotKey, (snapshotKey, previousSnapshotKey) => {
               </VRow>
             </div>
           </VWindowItem>
+          <VWindowItem value="data">
+            <div>
+              <VRow>
+                <VCol cols="12">
+                  <VSwitch
+                    v-model="SystemSettings.Advanced.DATA_CLEANUP_ENABLE"
+                    :label="t('setting.system.dataCleanupEnable')"
+                    :hint="t('setting.system.dataCleanupEnableHint')"
+                    persistent-hint
+                  />
+                </VCol>
+                <template v-if="SystemSettings.Advanced.DATA_CLEANUP_ENABLE">
+                  <VCol cols="12" md="6">
+                    <VTextField
+                      v-model.number="SystemSettings.Advanced.DATA_CLEANUP_MESSAGE_DAYS"
+                      :label="t('setting.system.dataCleanupMessageDays')"
+                      :hint="t('setting.system.dataCleanupMessageDaysHint')"
+                      persistent-hint
+                      min="0"
+                      type="number"
+                      :suffix="t('setting.system.day')"
+                      :rules="dataCleanupFieldRules"
+                      prepend-inner-icon="mdi-email-outline"
+                    />
+                  </VCol>
+                  <VCol cols="12" md="6">
+                    <VTextField
+                      v-model.number="SystemSettings.Advanced.DATA_CLEANUP_DOWNLOAD_HISTORY_DAYS"
+                      :label="t('setting.system.dataCleanupDownloadHistoryDays')"
+                      :hint="t('setting.system.dataCleanupDownloadHistoryDaysHint')"
+                      persistent-hint
+                      min="0"
+                      type="number"
+                      :suffix="t('setting.system.day')"
+                      :rules="dataCleanupFieldRules"
+                      prepend-inner-icon="mdi-download-circle-outline"
+                    />
+                  </VCol>
+                  <VCol cols="12" md="6">
+                    <VTextField
+                      v-model.number="SystemSettings.Advanced.DATA_CLEANUP_SITE_USERDATA_DAYS"
+                      :label="t('setting.system.dataCleanupSiteUserDataDays')"
+                      :hint="t('setting.system.dataCleanupSiteUserDataDaysHint')"
+                      persistent-hint
+                      min="0"
+                      type="number"
+                      :suffix="t('setting.system.day')"
+                      :rules="dataCleanupFieldRules"
+                      prepend-inner-icon="mdi-chart-line"
+                    />
+                  </VCol>
+                  <VCol cols="12" md="6">
+                    <VTextField
+                      v-model.number="SystemSettings.Advanced.DATA_CLEANUP_TRANSFER_HISTORY_DAYS"
+                      :label="t('setting.system.dataCleanupTransferHistoryDays')"
+                      :hint="t('setting.system.dataCleanupTransferHistoryDaysHint')"
+                      persistent-hint
+                      min="0"
+                      type="number"
+                      :suffix="t('setting.system.day')"
+                      :rules="dataCleanupFieldRules"
+                      prepend-inner-icon="mdi-swap-horizontal"
+                    />
+                  </VCol>
+                </template>
+              </VRow>
+            </div>
+          </VWindowItem>
           <VWindowItem value="log">
             <div>
               <VRow>
@@ -1986,77 +2054,6 @@ watch(currentLlmSnapshotKey, (snapshotKey, previousSnapshotKey) => {
               </VRow>
             </div>
           </VWindowItem>
-          <VWindowItem value="data">
-            <div>
-              <VRow>
-                <VCol cols="12">
-                  <VSwitch
-                    v-model="SystemSettings.Advanced.DATA_CLEANUP_ENABLE"
-                    :label="t('setting.system.dataCleanupEnable')"
-                    :hint="t('setting.system.dataCleanupEnableHint')"
-                    persistent-hint
-                  />
-                </VCol>
-                <VCol cols="12">
-                  <VAlert type="info" variant="tonal">
-                    {{ t('setting.system.downloadFilesCleanupNotice') }}
-                  </VAlert>
-                </VCol>
-                <VCol cols="12" md="6">
-                  <VTextField
-                    v-model.number="SystemSettings.Advanced.DATA_CLEANUP_MESSAGE_DAYS"
-                    :label="t('setting.system.dataCleanupMessageDays')"
-                    :hint="t('setting.system.dataCleanupMessageDaysHint')"
-                    persistent-hint
-                    min="0"
-                    type="number"
-                    :suffix="t('setting.system.day')"
-                    :rules="dataCleanupFieldRules"
-                    prepend-inner-icon="mdi-email-outline"
-                  />
-                </VCol>
-                <VCol cols="12" md="6">
-                  <VTextField
-                    v-model.number="SystemSettings.Advanced.DATA_CLEANUP_DOWNLOAD_HISTORY_DAYS"
-                    :label="t('setting.system.dataCleanupDownloadHistoryDays')"
-                    :hint="t('setting.system.dataCleanupDownloadHistoryDaysHint')"
-                    persistent-hint
-                    min="0"
-                    type="number"
-                    :suffix="t('setting.system.day')"
-                    :rules="dataCleanupFieldRules"
-                    prepend-inner-icon="mdi-download-circle-outline"
-                  />
-                </VCol>
-                <VCol cols="12" md="6">
-                  <VTextField
-                    v-model.number="SystemSettings.Advanced.DATA_CLEANUP_SITE_USERDATA_DAYS"
-                    :label="t('setting.system.dataCleanupSiteUserDataDays')"
-                    :hint="t('setting.system.dataCleanupSiteUserDataDaysHint')"
-                    persistent-hint
-                    min="0"
-                    type="number"
-                    :suffix="t('setting.system.day')"
-                    :rules="dataCleanupFieldRules"
-                    prepend-inner-icon="mdi-chart-line"
-                  />
-                </VCol>
-                <VCol cols="12" md="6">
-                  <VTextField
-                    v-model.number="SystemSettings.Advanced.DATA_CLEANUP_TRANSFER_HISTORY_DAYS"
-                    :label="t('setting.system.dataCleanupTransferHistoryDays')"
-                    :hint="t('setting.system.dataCleanupTransferHistoryDaysHint')"
-                    persistent-hint
-                    min="0"
-                    type="number"
-                    :suffix="t('setting.system.day')"
-                    :rules="dataCleanupFieldRules"
-                    prepend-inner-icon="mdi-swap-horizontal"
-                  />
-                </VCol>
-              </VRow>
-            </div>
-          </VWindowItem>
           <VWindowItem value="dev">
             <div>
               <VRow>