Merge pull request #7617 from Manishnemade12/Uncleaned

fix(navigation): clean up leaked global scroll listeners

Author: saurabhraghuvanshii

src/sections/Company/Brand/index.js

@@ -91,9 +91,15 @@ const easeOutCuaic = (t) => {
 const Brand = () => {
   const [scroll, setScroll] = useState(false);
   useEffect(() => {
-    window.addEventListener("scroll", () =>
-      window.scrollY > 510 ? setScroll(true) : setScroll(false)
-    );
+    const handleHeaderScroll = () => {
+      window.scrollY > 510 ? setScroll(true) : setScroll(false);
+    };
+
+    window.addEventListener("scroll", handleHeaderScroll);
+
+    return () => {
+      window.removeEventListener("scroll", handleHeaderScroll);
+    };
   }, []);
 
   const [visibleSection, setVisibleSection] = useState();

src/sections/General/Navigation/index.js

@@ -272,9 +272,15 @@ const Navigation = () => {
   }, [expand]);
 
   useEffect(() => {
-    window.addEventListener("scroll", () =>
-      window.scrollY > 50 ? setScroll(true) : setScroll(false),
-    );
+    const handleScroll = () => {
+      window.scrollY > 50 ? setScroll(true) : setScroll(false);
+    };
+
+    window.addEventListener("scroll", handleScroll);
+
+    return () => {
+      window.removeEventListener("scroll", handleScroll);
+    };
   }, []);
 
   const openDropDown = () => {