Initial logic for scroll view to required attachment#1058
Initial logic for scroll view to required attachment#1058Pratheek555 wants to merge 6 commits intoRocketChat:developfrom
Conversation
|
If a file isn’t in the current DOM, how would you navigate to that file attachment? Similar issues, like finding a starred message, often require implementing solutions with tools such as react-virtuoso, tanstack, or similar libraries. It would great for everyone to collaborate on this for a sustainable fix. Ref: #1059 |
|
Hey @Spiral-Memory I have been looking into using Tanstack query to solve this issue, will be working on a PR towards it. Please do let me know if you wanted anything different. Thank you! |
|
Hey @Spiral-Memory I have been working on the PR for sometime now and these are the major changes I have made. # Change - 1:
Virtualization.of.messages.mp4# Change - 2
FunctionLoopToLoadMessageStore.mp4
|
|
Please do let me know if this is what was expected and if there any changes to be made. As of now this does not go to the pinnedMessages inside a thread. I felt that as these are some fundamental changes once I get your approval of whether this is the right direction to proceed, then I would implement it. |
|
Thanks for the review I will push my changes after cleanly testing it. |
Brief Title
Implement initial scroll-to-message logic for navigating to attachments from the All Files view
Acceptance Criteria fulfillment
Fixes #1057
Video/Screenshots
EmbeddedChat._.Simple.-.Simple.Storybook.-.Brave.2026-01-07.13-46-01.mp4
PR Test Details
Note: The PR will be ready for live testing at https://rocketchat.github.io/EmbeddedChat/pulls/pr-<pr_number> after approval. Contributors are requested to replace
<pr_number>with the actual PR number.