Skip to content

Starting point for replacing the current timeline scroll bar with a virtualized list#32873

Closed
rbondesson wants to merge 33 commits intoelement-hq:developfrom
ZacksBot:refactor/scroll-panel
Closed

Starting point for replacing the current timeline scroll bar with a virtualized list#32873
rbondesson wants to merge 33 commits intoelement-hq:developfrom
ZacksBot:refactor/scroll-panel

Conversation

@rbondesson
Copy link
Copy Markdown
Contributor

@rbondesson rbondesson commented Mar 20, 2026

Checklist

Create labs setting and prepare for a new scroll panel

Summary

  • Created a new labs setting, feature_new_timeline, for activating the new rendering path in MessagePanel
  • Refactored MessagePanel and ScrollPanel to make a clean generic boundary for scroll panels

Changes

New component

  • Added a new scroll panel component, ´TimelineScrollPanel` which will act as a wrapper for the coming MVVM component
  • Duplicated the ScrollPanel unit-test suite for the TimelineScrollPanel for supporting the two rendering patchs

Codebase migration

  • No screenshots have been updated for element-web playwright tests

Test plan

  • Verify component renders correctly in the time line
  • Run unit tests for affected components in element-web
  • Run visual tests for affected components in element-web

Labs setting

image

@github-actions github-actions bot added the Z-Community-PR Issue is solved by a community member's PR label Mar 20, 2026
@rbondesson rbondesson requested a review from langleyd March 20, 2026 14:38
@rbondesson rbondesson added the T-Task Tasks for the team like planning label Mar 20, 2026
@langleyd
Copy link
Copy Markdown
Member

langleyd commented Apr 3, 2026

@rbondesson This seems to be working react virtuoso into the existing TImelinePanel and MessagePanel component tree, my suggestion was to replace them wholesale. So leave these old components largely untouched and build up the new timeline behind a labs flag. Evolving the existing one is an option with different tradeoffs but I think we could burn time on it.

This is what I had in mind -> #33042

@rbondesson rbondesson closed this Apr 9, 2026
@rbondesson
Copy link
Copy Markdown
Contributor Author

Created a new starting point as @langleyd suggested. #33073

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-Task Tasks for the team like planning Z-Community-PR Issue is solved by a community member's PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants