Skip to content

Commit a6b2a9c

Browse files
committed
fix memory leak in app header
1 parent 0589d8e commit a6b2a9c

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/components/AppHeader.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,12 @@ const AppHeader = () => {
3636
const sidebarShow = useSelector((state) => state.sidebarShow)
3737

3838
useEffect(() => {
39-
document.addEventListener('scroll', () => {
39+
const handleScroll = () => {
4040
headerRef.current &&
4141
headerRef.current.classList.toggle('shadow-sm', document.documentElement.scrollTop > 0)
42-
})
42+
}
43+
document.addEventListener('scroll', handleScroll)
44+
return () => document.removeEventListener('scroll', handleScroll)
4345
}, [])
4446

4547
return (

0 commit comments

Comments
 (0)