fix(useIntersection): return null if IntersectionObserver is not supp…

…orted
streamich · Jan 27, 2020 · 4f6d388 · 4f6d388
1 parent 3f8687e
commit 4f6d388
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/useIntersection.ts b/src/useIntersection.ts
@@ -7,7 +7,7 @@ const useIntersection = (
   const [intersectionObserverEntry, setIntersectionObserverEntry] = useState<IntersectionObserverEntry | null>(null);
 
   useEffect(() => {
-    if (ref.current) {
+    if (ref.current && typeof IntersectionObserver === 'function') {
       const handler = (entries: IntersectionObserverEntry[]) => {
         setIntersectionObserverEntry(entries[0]);
       };

diff --git a/tests/useIntersection.test.tsx b/tests/useIntersection.test.tsx
@@ -78,6 +78,14 @@ describe('useIntersection', () => {
     expect(result.current).toEqual(null);
   });
 
+  it('should return null if IntersectionObserver is not supported', () => {
+    targetRef = createRef();
+    targetRef.current = document.createElement('div');
+    delete window.IntersectionObserver;
+
+    expect(() => renderHook(() => useIntersection(targetRef, {}))).not.toThrow();
+  });
+
   it('should disconnect an old IntersectionObserver instance when the ref changes', () => {
     targetRef = createRef();
     targetRef.current = document.createElement('div');