[fix] Clear timeout on destroy (#166)

If the component gets destroyed before the timeout is cleared, you'll be invoking a method on a node that should not exist anymore

Author: DiegoBM

content/tutorial/03-advanced-svelte/04-actions/02-adding-parameters-to-actions/app-a/src/lib/longpress.js

@@ -16,6 +16,7 @@ export function longpress(node, duration) {
 
 	return {
 		destroy() {
+			clearTimeout(timer);
 			node.removeEventListener('mousedown', handleMousedown);
 			node.removeEventListener('mouseup', handleMouseup);
 		}

content/tutorial/03-advanced-svelte/04-actions/02-adding-parameters-to-actions/app-b/src/lib/longpress.js

@@ -19,6 +19,7 @@ export function longpress(node, duration) {
 			duration = newDuration;
 		},
 		destroy() {
+			clearTimeout(timer);
 			node.removeEventListener('mousedown', handleMousedown);
 			node.removeEventListener('mouseup', handleMouseup);
 		}