Merge pull request #37 from Ashish-simpleCoder/feature/develop-hooks

Fix working with ref for use-outside-click hook

.changeset/wise-grapes-thank.md

@@ -0,0 +1,5 @@
+---
+'classic-react-hooks': minor
+---
+
+Fix working with ref for use-outside-click hook

src/lib/use-outside-click/index.test.tsx

@@ -53,6 +53,22 @@ describe('use-outside-click', () => {
       expect(removeSpy).toHaveBeenCalledTimes(1)
    })
 
+   it('should fire listener when clicked outside of target element when ref is provided', () => {
+      const div = document.createElement('div')
+      const ref = { current: div }
+      const listener = vi.fn()
+
+      renderHook(() => {
+         useOutsideClick(ref, listener)
+      })
+
+      const event = new Event('click')
+      document.dispatchEvent(event)
+
+      expect(listener).toHaveBeenCalledTimes(1)
+      expect(listener).toHaveBeenCalledWith(event)
+   })
+
    it('should fire listener when clicked outside of target element', () => {
       const div = document.createElement('div')
 

src/lib/use-outside-click/index.tsx

@@ -41,8 +41,8 @@ export default function useOutsideClick(target: Target, handler: (event: Documen
 
       if (
          node &&
-         ((node as Node).contains(event.target as Node) ||
-            ('current' in node && (node.current as Node).contains(event.target as Node)))
+         (('contains' in node && (node as Node).contains(event.target as Node)) ||
+            ('current' in node && 'contains' && (node.current as Node).contains(event.target as Node)))
       ) {
          return
       }