zoriya/flood
1
0
Fork 0
mirror of https://github.com/zoriya/flood.git synced 2026-06-05 03:39:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

NotificationsButton: re-fetch notification only when count > 0

Browse Source
This commit is contained in:
Jesse Chan
2020-11-10 20:56:18 +08:00
parent 971a54390f
commit 0d4a60f636
1 changed files with 6 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
client/src/javascript/components/sidebar/NotificationsButton.tsx
+6 -7
View File
@@ -71,7 +71,12 @@ class NotificationsButton extends Component<WrappedComponentProps, Notifications
constructor(props: WrappedComponentProps) { constructor(props: WrappedComponentProps) {
super(props); super(props);
reaction(() => NotificationStore.notificationCount, this.handleNotificationCountChange); reaction(
() => NotificationStore.notificationCount,
(count) => {
if (count.total > 0 && this.tooltipRef?.isOpen()) this.fetchNotifications();
},
);
this.state = { this.state = {
isLoading: false, isLoading: false,
@@ -225,12 +230,6 @@ class NotificationsButton extends Component<WrappedComponentProps, Notifications
FloodActions.clearNotifications(); FloodActions.clearNotifications();
}; };
handleNotificationCountChange = () => {
if (this.tooltipRef != null && this.tooltipRef.isOpen()) {
this.fetchNotifications();
}
};
handleNewerNotificationsClick = () => { handleNewerNotificationsClick = () => {
if (this.state.paginationStart - NOTIFICATIONS_PER_PAGE >= 0) { if (this.state.paginationStart - NOTIFICATIONS_PER_PAGE >= 0) {
this.setState((state) => { this.setState((state) => {