From c74033b9c08dcc024403d0a3193301113ab6ca55 Mon Sep 17 00:00:00 2001 From: Will Miao <13051207myq@gmail.com> Date: Wed, 6 Aug 2025 11:14:02 +0800 Subject: [PATCH] refactor: conditionally initialize managers in HeaderManager to avoid unnecessary setup on statistics page --- static/js/components/Header.js | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/static/js/components/Header.js b/static/js/components/Header.js index a4722a28..96aca73b 100644 --- a/static/js/components/Header.js +++ b/static/js/components/Header.js @@ -16,7 +16,9 @@ export class HeaderManager { this.filterManager = null; // Initialize appropriate managers based on current page - this.initializeManagers(); + if (this.currentPage !== 'statistics') { + this.initializeManagers(); + } // Set up common header functionality this.initializeCommonElements(); @@ -37,11 +39,8 @@ export class HeaderManager { this.searchManager = new SearchManager({ page: this.currentPage }); window.searchManager = this.searchManager; - // Initialize FilterManager for all page types that have filters - if (document.getElementById('filterButton')) { - this.filterManager = new FilterManager({ page: this.currentPage }); - window.filterManager = this.filterManager; - } + this.filterManager = new FilterManager({ page: this.currentPage }); + window.filterManager = this.filterManager; } initializeCommonElements() {