Skip to content

fix(desktop): stabilize members dialog scrolling - #6670

Open
matt2e wants to merge 1 commit into
mainfrom
scrolling-members-dialog
Open

fix(desktop): stabilize members dialog scrolling#6670
matt2e wants to merge 1 commit into
mainfrom
scrolling-members-dialog

Conversation

@matt2e

@matt2e matt2e commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Keep virtualized member rows measurable by removing content-visibility: auto from the measured row subtree.
  • Use the member card's 60px baseline as the virtualizer estimate while retaining deferred rendering for eager search and archived-member lists.
  • Cover large rosters with a regression test that checks stable scroll extent across the list and verifies the final member remains reachable.

Related issue

None found.

Testing

  • just ci
  • pnpm -C desktop build:e2e
  • pnpm -C desktop exec playwright test tests/e2e/channels.spec.ts --grep 'members sidebar virtualizes large channel rosters' --repeat-each=5

Before

members-before.mov

After

members-after.mov

Signed-off-by: Matt Toohey <contact@matttoohey.com>
@matt2e
matt2e requested a review from a team as a code owner August 24, 2026 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant