-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Closed
Labels
Issue/PR - TriageNew issue. Needs quick review to confirm validity and assign labels.New issue. Needs quick review to confirm validity and assign labels.enhancementNew feature or requestNew feature or request
Description
What problem does this proposed feature solve?
This should improve the performance of the chat window because we're not loading all DOM elements, it's super slow to scroll once you have a lot of messages.
Opening the chat is also slow because you need to wait everything to load
Describe the proposed solution in detail
Virtualise the message list so we don't need to load all messages all at once, which is not necessary since not all of them are rendered, most of the times it's only a few messages that needs to get rendered
Technical considerations or implementation details (optional)
Use something like virtua to virtualise the list. See comparison: https://github.com/inokawa/virtua#comparison
Describe alternatives considered (if any)
Additional Context & Mockups
Proposal Checklist
- I have searched existing Issues and Discussions to ensure this proposal is not a duplicate.
- This proposal is for a specific, actionable change intended for implementation (not a general idea).
- I understand that this proposal requires review and approval before any development work begins.
Are you interested in implementing this feature if approved?
- Yes, I would like to contribute to implementing this feature.
Metadata
Metadata
Assignees
Labels
Issue/PR - TriageNew issue. Needs quick review to confirm validity and assign labels.New issue. Needs quick review to confirm validity and assign labels.enhancementNew feature or requestNew feature or request
Type
Projects
Status
Done