Skip to content

fix(sessions): smooth conversation scroll-up jank #2960

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
richardsolomou merged 7 commits into from
Jun 28, 2026
Merged

fix(sessions): smooth conversation scroll-up jank #2960

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
5cfd33f
fix(sessions): smooth conversation scroll-up jank
richardsolomou Jun 27, 2026
80b441d
fix(sessions): deepen overscan to kill scroll-up jank past heavy rows
richardsolomou Jun 27, 2026
e444816
test(sessions): parameterize chip estimates; guard scroll-adjust assi…
richardsolomou Jun 27, 2026
581a1ce
docs(sessions): tighten scroll-jank fix comments
richardsolomou Jun 27, 2026
c040879
refactor(sessions): simplify scroll-up fix to the overscan bump
richardsolomou Jun 27, 2026
b2f06d1
Merge branch 'main' into posthog-code/fix-virtualized-scroll-up-jank
richardsolomou Jun 27, 2026
ae4f48f
Merge branch 'main' into posthog-code/fix-virtualized-scroll-up-jank
richardsolomou Jun 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion packages/ui/src/features/sessions/components/VirtualizedList.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,10 @@ export interface VirtualizedListHandle {

const AT_BOTTOM_THRESHOLD = 50;
const ESTIMATED_ROW_SIZE = 80;
const OVERSCAN = 6;
// Render rows well ahead so tall, async rows (markdown, code, diffs) measure and
// paint off-screen instead of shifting and stuttering as they enter view. 12
// erases the scroll-up shift empirically; higher only adds DOM cost.
const OVERSCAN = 12;
// A real upward drift, not a 1-frame measure transient: the DOM bottom sits
// this far below the viewport. Well above any single append's measure gap.
const FAR_DRIFT_THRESHOLD = 400;
Expand Down
Loading