feat(useEventListener): Add null type in first parameter (#79)

insoftlabs · Dec 9, 2023 · 5445202 · 5445202
1 parent c705e87
commit 5445202
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/.changeset/wild-socks-type.md b/.changeset/wild-socks-type.md
@@ -0,0 +1,5 @@
+---
+'@raddix/use-event-listener': minor
+---
+
+Add null type in first parameter of useEventListener
diff --git a/packages/hooks/use-event-listener/src/index.ts b/packages/hooks/use-event-listener/src/index.ts
@@ -1,11 +1,10 @@
 import { type RefObject, useEffect, useRef } from 'react';
 
-export const _window = typeof window !== 'undefined' ? window : global.window;
-export const _document =
-  typeof document !== 'undefined' ? document : global.document;
+export const _window = typeof window !== 'undefined' ? window : null;
+export const _document = typeof document !== 'undefined' ? document : null;
 
 export const useEventListener = <T extends keyof HTMLElementEventMap>(
-  target: RefObject<EventTarget> | EventTarget,
+  target: RefObject<EventTarget> | EventTarget | null,
   eventType: T,
   callback: (event: HTMLElementEventMap[T]) => void,
   options?: boolean | AddEventListenerOptions
@@ -17,6 +16,7 @@ export const useEventListener = <T extends keyof HTMLElementEventMap>(
   }, [callback]);
 
   useEffect(() => {
+    if (target === null) return;
     const targetElement = 'current' in target ? target.current : target;
 
     if (!targetElement?.addEventListener) return;