Merge pull request #4 from chungchunwang/main

Resolves bug where interval continues even on new page.
MartinPicc · Dec 6, 2022 · da784ed · da784ed
2 parents cb200ca + d7a0d8c
commit da784ed
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "Interval",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "Set time interval to trigger actions.",
   "license": "MIT",
   "svelte": "index.js",

diff --git a/src/lib/Interval.svelte b/src/lib/Interval.svelte
@@ -1,16 +1,20 @@
 <script>
-  import { createEventDispatcher } from "svelte";
+  import { createEventDispatcher, onDestroy } from "svelte";
   import stopwatchSvg from './icons/stopwatch.svg';
 
   const dispatch = createEventDispatcher();
   export let interval;
   export let display = true;
   export let text = "Interval";
 
+  let intervalID = null;
+
   if (interval > 0) {
-    window.setInterval(() => { dispatch("trigger"); }, interval * 1000);
+    intervalID = window.setInterval(() => { dispatch("trigger"); }, interval * 1000);
   }
-
+  onDestroy(() => {
+    if(intervalID) window.clearInterval(intervalID);
+  });
 </script>
 
 {#if display}