matty/ncd-fe
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

nicolium: possibly improve notifications behavior

Browse Source 
Signed-off-by: nicole mikołajczyk <git@mkljczk.pl>
This commit is contained in:
nicole mikołajczyk
2026-02-25 21:33:25 +01:00
parent f366c8a8d1
commit ddbf26d9fd
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
packages/pl-fe/src/columns/notifications.tsx
View File

@ -225,6 +225,7 @@ const NotificationsColumn: React.FC<INotificationsColumn> = ({ multiColumn }) =>
}, [notifications, topNotification]);
const hasMore = hasNextPage ?? false;
const isFirstRender = useRef(true);
const node = useRef<VirtuosoHandle | null>(null);
const scrollableContentRef = useRef<Array<React.JSX.Element> | null>(null);
@ -266,7 +267,7 @@ const NotificationsColumn: React.FC<INotificationsColumn> = ({ multiColumn }) =>
};
const handleDequeueNotifications = useCallback(() => {
setTopNotification(undefined);
setTopNotification(notifications[0]?.most_recent_notification_id);
markNotificationsRead(notifications[0]?.most_recent_notification_id);
}, [notifications, markNotificationsRead]);
@ -296,6 +297,10 @@ const NotificationsColumn: React.FC<INotificationsColumn> = ({ multiColumn }) =>
}, []);
useEffect(() => {
if (isFirstRender.current) {
isFirstRender.current = false;
return;
}
setTopNotification(undefined);
}, [activeFilter]);