moon
/
soapbox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Only mark last message as read if needed

This commit is contained in:
Chewbacca 2022-11-09 10:16:04 -05:00
parent e7e121e78f
commit d637626838
1 changed files with 16 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
app/soapbox/features/chats/components/chat-message-list.tsx
View File

@ -74,7 +74,9 @@ const ChatMessageList: React.FC<IChatMessageList> = ({ chat }) => {
const features = useFeatures(); const features = useFeatures();
const lastReadMessageDateString = chat.latest_read_message_by_account.find((latest) => latest.id === chat.account.id)?.date; const lastReadMessageDateString = chat.latest_read_message_by_account.find((latest) => latest.id === chat.account.id)?.date;
const myLastReadMessageDateString = chat.latest_read_message_by_account.find((latest) => latest.id === account?.id)?.date;
const lastReadMessageTimestamp = lastReadMessageDateString ? new Date(lastReadMessageDateString) : null; const lastReadMessageTimestamp = lastReadMessageDateString ? new Date(lastReadMessageDateString) : null;
const myLastReadMessageTimestamp = myLastReadMessageDateString ? new Date(myLastReadMessageDateString) : null;
const node = useRef<VirtuosoHandle>(null); const node = useRef<VirtuosoHandle>(null);
const [firstItemIndex, setFirstItemIndex] = useState(START_INDEX - 20); const [firstItemIndex, setFirstItemIndex] = useState(START_INDEX - 20);
@ -370,10 +372,21 @@ const ChatMessageList: React.FC<IChatMessageList> = ({ chat }) => {
}; };
useEffect(() => { useEffect(() => {
const lastMessage = formattedChatMessages.pop(); const lastMessage = formattedChatMessages[formattedChatMessages.length - 1];
const lastMessageId = lastMessage?.id; if (!lastMessage) {
return;
}
if (lastMessageId && !lastMessage.pending) { const lastMessageId = lastMessage.id;
const isMessagePending = lastMessage.pending;
const isAlreadyRead = myLastReadMessageTimestamp ? myLastReadMessageTimestamp >= new Date(lastMessage.created_at) : false;
/**
* Only "mark the message as read" if..
* 1) it is not pending and
* 2) it has not already been read
*/
if (!isMessagePending && !isAlreadyRead) {
markChatAsRead(lastMessageId); markChatAsRead(lastMessageId);
} }
}, [formattedChatMessages.length]); }, [formattedChatMessages.length]);