fix: remove debounce on beforeDestroy to not have a script that is triggered despite the element being removed from the dom

Author: ga-devfront

_dev/src/ts/components/LogsViewer.ts

@@ -45,7 +45,7 @@ export default class LogsViewer extends ComponentAbstract implements DomLifecycl
   }
 
   public mount = () => {
-    this.#logsScroll.addEventListener('scroll', this.#debouncedRefreshView);
+    this.#logsScroll.addEventListener('scroll', this.#debouncedRefreshView.debounced);
 
     // delay needed because of side menu toggle on small screens
     // we set width to prevent the modification of the height of
@@ -57,8 +57,9 @@ export default class LogsViewer extends ComponentAbstract implements DomLifecycl
 
   public beforeDestroy = () => {
     logStore.clearLogs();
-    this.#logsScroll.removeEventListener('scroll', this.#debouncedRefreshView);
+    this.#logsScroll.removeEventListener('scroll', this.#debouncedRefreshView.debounced);
     this.#logsSummary.removeEventListener('click', this.#handleLinkEvent);
+    this.#debouncedRefreshView.cancel();
   };
 
   /**

_dev/src/ts/utils/logsUtils.ts

@@ -50,18 +50,22 @@ export function parseLogWithSeverity(log: string): LogEntry {
 
 /**
  * @public
- * @param {T} func
- * @param {number} wait
- * @return {(...args: Parameters<T>) => void}
- * @description
+ * @template T
+ * @param {T} func - The function to debounce.
+ * @param {number} wait - The delay in milliseconds before the function is executed.
+ * @return {(...args: Parameters<T>) => void & { clear: () => void }} - A debounced function
+ * that delays the execution of `func` and provides a `clear` method to cancel any pending execution.
+ * @description Creates a debounced version of the given function, ensuring it is executed
+ * only after the specified delay has elapsed since the last invocation.
+ * The returned function also includes a `clear` method to cancel any pending executions.
  */
 export function debounce<T extends Procedure>(
   func: T,
   wait: number
-): (...args: Parameters<T>) => void {
+): { debounced: (...args: Parameters<T>) => void; cancel: () => void } {
   let timeoutId: ReturnType<typeof setTimeout> | undefined;
 
-  return (...args: Parameters<T>): void => {
+  const debounced = (...args: Parameters<T>): void => {
     if (timeoutId) {
       clearTimeout(timeoutId);
     }
@@ -70,6 +74,15 @@ export function debounce<T extends Procedure>(
       func(...args);
     }, wait);
   };
+
+  const cancel = (): void => {
+    if (timeoutId) {
+      clearTimeout(timeoutId);
+      timeoutId = undefined;
+    }
+  };
+
+  return { debounced, cancel };
 }
 
 /**