refactor: conditionally initialize managers in HeaderManager to avoid unnecessary setup on statistics page

This commit is contained in:
Will Miao
2025-08-06 11:14:02 +08:00
parent d2b21d27bb
commit c74033b9c0

View File

@@ -16,7 +16,9 @@ export class HeaderManager {
this.filterManager = null; this.filterManager = null;
// Initialize appropriate managers based on current page // Initialize appropriate managers based on current page
if (this.currentPage !== 'statistics') {
this.initializeManagers(); this.initializeManagers();
}
// Set up common header functionality // Set up common header functionality
this.initializeCommonElements(); this.initializeCommonElements();
@@ -37,12 +39,9 @@ export class HeaderManager {
this.searchManager = new SearchManager({ page: this.currentPage }); this.searchManager = new SearchManager({ page: this.currentPage });
window.searchManager = this.searchManager; window.searchManager = this.searchManager;
// Initialize FilterManager for all page types that have filters
if (document.getElementById('filterButton')) {
this.filterManager = new FilterManager({ page: this.currentPage }); this.filterManager = new FilterManager({ page: this.currentPage });
window.filterManager = this.filterManager; window.filterManager = this.filterManager;
} }
}
initializeCommonElements() { initializeCommonElements() {
// Handle theme toggle // Handle theme toggle