Skip to content

Commit 62701cd

Browse files
committed
fix: correctly remove event listener by passing same handler reference
1 parent 4220e7f commit 62701cd

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

hooks/useEventListener.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,10 @@ export const useEventListener = ({
99
handler,
1010
}: UseEventListenerProps) => {
1111
useEffect(() => {
12-
window.addEventListener(eventName, event => {
13-
handler(event)
14-
})
12+
window.addEventListener(eventName, handler)
1513

1614
return () => {
17-
window.removeEventListener(eventName, () => {})
15+
window.removeEventListener(eventName, handler)
1816
}
1917
}, [eventName, handler])
2018

0 commit comments

Comments
 (0)