fix: removeEvent not work

Simon he · Simon he · commit daf569d49363 · 2022-10-14T20:35:21.000+08:00
diff --git a/src/addEventListener.ts b/src/addEventListener.ts
@@ -56,12 +56,13 @@ export function addEventListener(target: Window | Document | Element | string, e
       event(e)
     }
     target.addEventListener(eventName, eventFunction, useCapture)
-    stop = () => (target as Element).removeEventListener(eventName, event, useCapture)
+    stop = () => (target as Element).removeEventListener(eventName, eventFunction, useCapture)
+    if (stopped)
+      stop?.()
     hasMounted = true
   }
   return () => {
-    if (!stop)
-      return stopped = true
-    setTimeout(stop)
+    if (!stop) return stopped = true
+    stop?.()
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -56,12 +56,13 @@ export function addEventListener(target: Window \| Document \| Element \| string, e`
`56`	`56`	`event(e)`
`57`	`57`	`}`
`58`	`58`	`target.addEventListener(eventName, eventFunction, useCapture)`
`59`		`- stop = () => (target as Element).removeEventListener(eventName, event, useCapture)`
	`59`	`+ stop = () => (target as Element).removeEventListener(eventName, eventFunction, useCapture)`
	`60`	`+ if (stopped)`
	`61`	`+ stop?.()`
`60`	`62`	`hasMounted = true`
`61`	`63`	`}`
`62`	`64`	`return () => {`
`63`		`- if (!stop)`
`64`		`- return stopped = true`
`65`		`- setTimeout(stop)`
	`65`	`+ if (!stop) return stopped = true`
	`66`	`+ stop?.()`
`66`	`67`	`}`
`67`	`68`	`}`