From 4eda92f25c8e9ec16ec4f3677728ec623ab84d51 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E8=98=AD?= <81747598+lan-yonghui@users.noreply.github.com>
Date: Wed, 22 Apr 2026 16:53:24 +0800
Subject: [PATCH] refactor: simplify pagination logic and update theme change
 listener (#12564)

---
 .../src/components/complex-table/index.vue    |  8 +---
 frontend/src/global/use-theme.ts              | 11 +++---
 .../code-editor/history/index.vue             | 37 +++----------------
 3 files changed, 12 insertions(+), 44 deletions(-)

diff --git a/frontend/src/components/complex-table/index.vue b/frontend/src/components/complex-table/index.vue
index b4372b680..3579e2854 100644
--- a/frontend/src/components/complex-table/index.vue
+++ b/frontend/src/components/complex-table/index.vue
@@ -213,13 +213,7 @@ const responsivePaginationLayout = computed(() => {
 });
 
 const responsivePagerCount = computed(() => {
-    if (mobile.value || props.paginationConfig?.small) {
-        return 5;
-    }
-    if (paginationWidth.value < 520) {
-        return 3;
-    }
-    if (paginationWidth.value < 720) {
+    if (mobile.value || props.paginationConfig?.small || paginationWidth.value < 720) {
         return 5;
     }
     return 7;
diff --git a/frontend/src/global/use-theme.ts b/frontend/src/global/use-theme.ts
index 541664e4d..e072eb9a2 100644
--- a/frontend/src/global/use-theme.ts
+++ b/frontend/src/global/use-theme.ts
@@ -1,4 +1,4 @@
-import { onUnmounted } from 'vue';
+import { getCurrentScope, onScopeDispose } from 'vue';
 import { GlobalStore } from '@/store';
 import { setPrimaryColor } from '@/utils/theme';
 
@@ -27,7 +27,6 @@ export const useTheme = () => {
         }
     };
 
-    // Listen for OS theme changes when theme is set to "auto"
     const mediaQuery = window.matchMedia('(prefers-color-scheme: dark)');
     const onSystemThemeChange = () => {
         const globalStore = GlobalStore();
@@ -36,9 +35,11 @@ export const useTheme = () => {
         }
     };
     mediaQuery.addEventListener('change', onSystemThemeChange);
-    onUnmounted(() => {
-        mediaQuery.removeEventListener('change', onSystemThemeChange);
-    });
+    if (getCurrentScope()) {
+        onScopeDispose(() => {
+            mediaQuery.removeEventListener('change', onSystemThemeChange);
+        });
+    }
 
     return {
         switchTheme,
diff --git a/frontend/src/views/host/file-management/code-editor/history/index.vue b/frontend/src/views/host/file-management/code-editor/history/index.vue
index b78f41145..917a22c35 100644
--- a/frontend/src/views/host/file-management/code-editor/history/index.vue
+++ b/frontend/src/views/host/file-management/code-editor/history/index.vue
@@ -102,7 +102,7 @@
 
                     <div class="table-box history-table-box">
                         <div class="history-table-wrap">
-                            <el-table
+                            <ComplexTable
                                 :pagination-config="paginationConfig"
                                 :data="historyItems"
                                 v-loading="historyLoading"
@@ -142,7 +142,7 @@
                                         {{ computeSize(row.contentSize) }}
                                     </template>
                                 </el-table-column>
-                                <el-table-column :label="$t('commons.table.operate')" min-width="110" fixed="right">
+                                <el-table-column :label="$t('commons.table.operate')" min-width="150" fixed="right">
                                     <template #default="{ row }">
                                         <el-button link type="primary" @click.stop="openHistoryRecord(row)">
                                             {{ $t('commons.button.view') }}
@@ -152,23 +152,8 @@
                                         </el-button>
                                     </template>
                                 </el-table-column>
-                            </el-table>
+                            </ComplexTable>
                         </div>
-
-                        <el-pagination
-                            v-model:current-page="pagination.currentPage"
-                            v-model:page-size="pagination.pageSize"
-                            :page-sizes="[5, 10, 20, 50, 100, 200, 500]"
-                            :size="mobile || paginationConfig.small ? 'small' : 'default'"
-                            :layout="
-                                mobile || paginationConfig.small
-                                    ? 'total, prev, pager, next'
-                                    : 'total, sizes, prev, pager, next, jumper'
-                            "
-                            :total="pagination.total"
-                            @current-change="handlePageChange"
-                            @size-change="handleSizeChange"
-                        />
                     </div>
                 </el-card>
 
@@ -235,8 +220,8 @@ import { Setting } from '@/api/interface/setting';
 import { loadMonacoLanguageSupport, setupMonacoEnvironment } from '@/utils/monaco';
 import { ElMessageBox, type FormInstance, type FormRules } from 'element-plus';
 import { Languages } from '@/global/mimetype';
-import { GlobalStore } from '@/store';
 import i18n from '@/lang';
+import ComplexTable from '@/components/complex-table/index.vue';
 
 type MonacoEditorApi = typeof import('monaco-editor/esm/vs/editor/editor.api');
 
@@ -258,10 +243,7 @@ const historySettingFormRef = ref<FormInstance>();
 const scope = ref<'current' | 'all'>('current');
 const operationFilter = ref('');
 const activeCollapse = ref([]);
-const globalStore = GlobalStore();
-const mobile = computed(() => {
-    return globalStore.isMobile();
-});
+
 const paginationConfig = reactive({
     cacheSizeKey: 'file-history-page-size',
     currentPage: 1,
@@ -602,15 +584,6 @@ const handleScopeChange = async () => {
     await loadHistoryList(true);
 };
 
-const handlePageChange = async (page: number) => {
-    pagination.value.currentPage = page;
-    await loadHistoryList();
-};
-
-const handleSizeChange = async () => {
-    await loadHistoryList(true);
-};
-
 const handleClose = () => {
     drawerVisible.value = false;
     disposeDiffEditor();