From a6b2a9c99619a7f866077c2fa72d984d37b21bd2 Mon Sep 17 00:00:00 2001 From: codomposer Date: Wed, 29 Oct 2025 10:36:30 -0400 Subject: [PATCH] fix memory leak in app header --- src/components/AppHeader.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/components/AppHeader.js b/src/components/AppHeader.js index b10bd7e124..900c0675b0 100644 --- a/src/components/AppHeader.js +++ b/src/components/AppHeader.js @@ -36,10 +36,12 @@ const AppHeader = () => { const sidebarShow = useSelector((state) => state.sidebarShow) useEffect(() => { - document.addEventListener('scroll', () => { + const handleScroll = () => { headerRef.current && headerRef.current.classList.toggle('shadow-sm', document.documentElement.scrollTop > 0) - }) + } + document.addEventListener('scroll', handleScroll) + return () => document.removeEventListener('scroll', handleScroll) }, []) return (