Skip to content

fix: preserve scroll position when streaming finishes#70

Merged
quiet-node merged 1 commit intomainfrom
worktree-buzzing-squishing-clarke
Apr 10, 2026
Merged

fix: preserve scroll position when streaming finishes#70
quiet-node merged 1 commit intomainfrom
worktree-buzzing-squishing-clarke

Conversation

@quiet-node
Copy link
Copy Markdown
Owner

Summary

  • Only re-enable auto-scroll when a user message is added (new prompt sent), not when an assistant message is finalized after streaming completes
  • Previously, the end of streaming would snap the view to the bottom even if the user had scrolled up to read earlier content
  • Added test coverage for the assistant-message-finalized scenario

Test plan

  • All 463 frontend tests pass
  • bun run validate-build passes with zero warnings/errors
  • Manual: send a long prompt, scroll up while streaming, verify view stays put after streaming ends
  • Manual: send a new message after scrolling up, verify auto-scroll re-engages

🤖 Generated with Claude Code

Only re-enable auto-scroll when a user message is added (new prompt),
not when an assistant message is finalized. Previously, the completion
of streaming would snap the view to the bottom even if the user had
scrolled up to read earlier content.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Logan Nguyen <lg.131.dev@gmail.com>
@quiet-node quiet-node merged commit 4254ea2 into main Apr 10, 2026
3 checks passed
@quiet-node quiet-node deleted the worktree-buzzing-squishing-clarke branch April 10, 2026 01:13
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