feat: Jump to last message WPB-6518 #17386
Conversation
|
|
| isVisible: boolean; | ||
| } | ||
|
|
||
| export interface LastMessageTrackerProps extends HTMLProps<HTMLElement> { |
There was a problem hiding this comment.
This should be a separate component as we want to avoid the entire content of Conversation to re-render each time messages are scrolled and messages visibility changes
| @@ -88,6 +89,7 @@ export const Message: React.FC<MessageParams & {scrollTo?: ScrollToElement}> = p | |||
| isHighlighted, | |||
| hideHeader, | |||
| onVisible, | |||
| onVisibilityChange, | |||
There was a problem hiding this comment.
I know this is weird, but as isVisible was implemented till now, it would be triggered only the first time (and if it's added as the item of the list one by one, and then disappear from the visible area - it will still be called). I don't mind merging them at least on the Message and the level of its usage, but I want to hear your opinions on how to do it the best
svitovyda
force-pushed
the
feat/WPB-6518-jump_to_last_message
branch
from
70bff5b
to
caec31b
Compare
…sage; Mark all the conversation as read on the "jump to last message" button click; Reload the conversation on the "jump to last message" click if the latest conversation messages were not load;
svitovyda
force-pushed
the
feat/WPB-6518-jump_to_last_message
branch
from
caec31b
to
691c0bc
Compare
|
Description
By the product/design team request, the user should jump to the very last message in the conversation, marking all the messages as read
Screenshots/Screencast (for UI changes)
Checklist
Important details for the reviewers
(Delete this section if unnecessary)