From 0bf369907bea93af2e57ee65b2cb17df4b0463c6 Mon Sep 17 00:00:00 2001 From: "Janus C. H. Knudsen" Date: Thu, 2 Oct 2025 16:05:11 +0200 Subject: [PATCH] Refactors all-day event overflow toggle. Simplifies the all-day event overflow toggle logic by using distinct class names and avoiding direct class switching, improving code readability and maintainability. --- src/managers/AllDayManager.ts | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/src/managers/AllDayManager.ts b/src/managers/AllDayManager.ts index d4ec4d0..f80e891 100644 --- a/src/managers/AllDayManager.ts +++ b/src/managers/AllDayManager.ts @@ -524,14 +524,17 @@ export class AllDayManager { this.isExpanded = !this.isExpanded; this.checkAndAnimateAllDayHeight(); - - let elements = document.querySelectorAll('swp-allday-container swp-event : hasclass max-event-overflow-hide'); - elements.forEach(element: HTMLElement => { - - element.classList.switch('max-event-overflow-hide', 'max-event-overflow-show'); - - } - + let elements = document.querySelectorAll('swp-allday-container swp-event.max-event-overflow-hide, swp-allday-container swp-event.max-event-overflow-show'); + elements.forEach((element) => { + if (element.classList.contains('max-event-overflow-hide')) { + element.classList.remove('max-event-overflow-hide'); + element.classList.add('max-event-overflow-show'); + } else if (element.classList.contains('max-event-overflow-show')) { + element.classList.remove('max-event-overflow-show'); + element.classList.add('max-event-overflow-hide'); + } + }); + } /** * Count number of events in a specific column using ColumnBounds */