Fixed navbar section warning

swapnilnarad2000 · Feb 5, 2024 · 943df45 · 943df45
1 parent 319e797
commit 943df45
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/Components/NavbarSection.js b/src/Components/NavbarSection.js
@@ -2,6 +2,7 @@ import Container from 'react-bootstrap/Container';
 import Nav from 'react-bootstrap/Nav';
 import Navbar from 'react-bootstrap/Navbar';
 import React, { useState, useEffect } from 'react';
+import { useCallback } from 'react';
 
 import Image from 'react-bootstrap/Image';
 
@@ -16,7 +17,7 @@ let NavbarSection = () => {
   const [overlay, setOverlay] = useState(false);
   const [lastScrollY, setLastScrollY] = useState(0);
 
-  const controlNavbar = () => {
+  const controlNavbar = useCallback(() => {
     if(window.scrollY === 0){
       setOverlay(false);
     }
@@ -29,7 +30,7 @@ let NavbarSection = () => {
 
     // remember current page location to use in the next move
     setLastScrollY(window.scrollY);
-  };
+  }, [lastScrollY]);
 
   useEffect(() => {
     window.addEventListener('scroll', controlNavbar);
@@ -38,7 +39,7 @@ let NavbarSection = () => {
     return () => {
       window.removeEventListener('scroll', controlNavbar);
     };
-  }, [lastScrollY]);
+  }, [lastScrollY, controlNavbar]);
 
 
   return (