fix: clear timeout within useEffect hook

Signed-off-by: abhijithvijayan <[email protected]>
thedevs-network · Apr 4, 2021 · 84f4af6 · 84f4af6
1 parent 8857890
commit 84f4af6
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/source/History/Table.tsx b/source/History/Table.tsx
@@ -43,10 +43,18 @@ const Table: React.FC = () => {
 
   // reset copy message
   useEffect(() => {
-    setTimeout(() => {
+    let timer: NodeJS.Timeout | null = null;
+
+    timer = setTimeout(() => {
       setCopied(false);
       // reset selected id from context
     }, 1300);
+
+    return (): void => {
+      if (timer) {
+        clearTimeout(timer);
+      }
+    };
   }, [copied]);
 
   function handleCopyToClipboard(selectedItemId: string): void {

diff --git a/source/Popup/ResponseBody.tsx b/source/Popup/ResponseBody.tsx
@@ -38,9 +38,17 @@ const ResponseBody: React.FC = () => {
 
   // reset copy message
   useEffect(() => {
-    setTimeout(() => {
+    let timer: NodeJS.Timeout | null = null;
+
+    timer = setTimeout(() => {
       setCopied(false);
     }, 1300);
+
+    return (): void => {
+      if (timer) {
+        clearTimeout(timer);
+      }
+    };
   }, [copied]);
 
   return (