From 9a0ea3ac43c277d7cece0327d264dd2ff86a3d2d Mon Sep 17 00:00:00 2001 From: Manav Rathi Date: Mon, 23 Sep 2024 16:59:39 +0530 Subject: [PATCH] Match the event name --- .../Collections/CollectionListBar.tsx | 20 ++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/web/apps/photos/src/components/Collections/CollectionListBar.tsx b/web/apps/photos/src/components/Collections/CollectionListBar.tsx index 8a43a6ebb9..148d50d243 100644 --- a/web/apps/photos/src/components/Collections/CollectionListBar.tsx +++ b/web/apps/photos/src/components/Collections/CollectionListBar.tsx @@ -79,6 +79,7 @@ export interface CollectionListBarProps { onSelectPerson: (person: Person) => void; } +// TODO-Cluster Rename me to GalleryBar export const CollectionListBar: React.FC = ({ mode, setMode, @@ -90,7 +91,7 @@ export const CollectionListBar: React.FC = ({ setCollectionListSortBy, people, activePerson, - // onSelectPerson + onSelectPerson }) => { const windowSize = useWindowSize(); const isMobile = useIsMobileWidth(); @@ -113,18 +114,19 @@ export const CollectionListBar: React.FC = ({ }; useEffect(() => { - if (!listWrapperRef.current) { - return; - } - // Add event listener - listWrapperRef.current?.addEventListener("scroll", updateScrollObj); + if (!listWrapperRef.current) return; - // Call handler right away so state gets updated with initial window size + // Add event listener. + listWrapperRef.current.addEventListener("scroll", updateScrollObj); + + // Call handler right away so that state gets updated with initial + // window size. updateScrollObj(); - // Remove event listener on cleanup + + // Remove event listener on cleanup. return () => listWrapperRef.current?.removeEventListener( - "resize", + "scroll", updateScrollObj, ); }, [listWrapperRef.current]);