fix: remove unnecessary destroy

Closes #3

Author: mikededo

apps/finance/src/lib/domain/budget/actions.svelte.ts

@@ -129,10 +129,4 @@ export const useUpDownArrows: Action<HTMLElement, string> = (node, prefix) => {
   };
 
   node.addEventListener('keydown', onKeydown);
-
-  return {
-    destroy() {
-      node.removeEventListener('keydown', onKeydown);
-    }
-  };
 };

apps/finance/src/lib/domain/page/table/entry.svelte

@@ -89,12 +89,6 @@
         };
 
         node.addEventListener('keydown', onKeydown);
-
-        return {
-            destroy() {
-                node.removeEventListener('keydown', onKeydown);
-            }
-        };
     };
 
     const onInternalClickAway = (event: MouseEvent) => {

apps/finance/src/lib/domain/tags/menu/autocomplete-menu-list.ts

@@ -30,11 +30,4 @@ export const useInputMenuOption: Action<HTMLElement, UseMenuOptionArgs> = (
 
   node.addEventListener('keydown', onKeydown);
   node.addEventListener('mousedown', onOptionClick);
-
-  return {
-    destroy() {
-      node.removeEventListener('keydown', onKeydown);
-      node.removeEventListener('mousedown', onOptionClick);
-    }
-  };
 };

packages/actions/src/date-mask.ts

@@ -46,11 +46,4 @@ export const useDateMask: Action = (node) => {
 
   node.addEventListener('input', onInput, true);
   node.addEventListener('keydown', onKeydown);
-
-  return {
-    destroy() {
-      node.removeEventListener('input', onInput, true);
-      node.removeEventListener('keydown', onKeydown);
-    }
-  };
 };

packages/actions/src/use-money-mask.ts

@@ -28,14 +28,6 @@ export const useMoneyMask: Action = (node) => {
   };
 
   node.addEventListener('input', onInput, true);
-
   // Register also the prefix of the currency
-  const prefix = usePrefix(node, '€ ');
-
-  return {
-    destroy() {
-      node.removeEventListener('input', onInput, true);
-      prefix.destroy?.();
-    }
-  };
+  usePrefix(node, '€ ');
 };

packages/actions/src/use-percentage-mask.ts

@@ -22,13 +22,5 @@ export const usePercentageMask: Action = (node) => {
   };
 
   node.addEventListener('input', onInput, true);
-
-  const prefix = usePrefix(node, '% ');
-
-  return {
-    destroy() {
-      node.removeEventListener('input', onInput, true);
-      prefix.destroy?.();
-    }
-  };
+  usePrefix(node, '% ');
 };

packages/actions/src/use-prefix.ts

@@ -1,6 +1,4 @@
-import type { ActionReturn } from 'svelte/action';
-
-export const usePrefix = (node: HTMLInputElement, prefix: string): ActionReturn => {
+export const usePrefix = (node: HTMLInputElement, prefix: string): void => {
   const onInput = () => {
     if (!node.value.startsWith(prefix) && node.value) {
       node.value = prefix + node.value;
@@ -11,10 +9,4 @@ export const usePrefix = (node: HTMLInputElement, prefix: string): ActionReturn
 
   node.addEventListener('input', onInput, true);
   onInput();
-
-  return {
-    destroy() {
-      node.removeEventListener('input', onInput, true);
-    }
-  };
 };

packages/actions/src/use-trap-focus.ts

@@ -19,10 +19,4 @@ export const useTrapFocus: Action = (node) => {
   };
 
   node.addEventListener('keydown', onKeydown);
-
-  return {
-    destroy() {
-      node.removeEventListener('keydown', onKeydown);
-    }
-  };
 };

packages/ui/src/drawer/drawer.svelte

@@ -62,12 +62,6 @@
         };
 
         node.addEventListener('click', onClick);
-
-        return {
-            destroy() {
-                node.removeEventListener('click', onClick);
-            }
-        };
     };
 
     $effect(() => {

packages/ui/src/tabs/tabs.svelte

@@ -30,11 +30,6 @@
         };
 
         node.addEventListener('click', onClick);
-        return {
-            destroy() {
-                node.removeEventListener('click', onClick);
-            }
-        };
     };
 
     $effect(() => {