fix(mothership): keep chat pinned to bottom across multi-line input resizes#5711
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview Prompt editor: During autosize, the scroller’s height is frozen at its current pixel size while the textarea briefly collapses to Mothership chat: Adds a Reviewed by Cursor Bugbot for commit 56574c8. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
@greptile review |
Greptile SummaryThis PR keeps the mothership chat pinned during input and container resizing. The main changes are:
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (1): Last reviewed commit: "fix(mothership): keep chat pinned to bot..." | Re-trigger Greptile |
Summary
height:'auto'→ readscrollHeight→ set px) forces a synchronous layout with the textarea collapsed; during that transient layout the chat scroll container grows and the browser natively clamps the pinnedscrollTopdown by the input's grown height (no JS write, invisible to ResizeObserver since the net size change is zero)Type of Change
Testing
Reproduced and verified in the running app with instrumented scroll tracing (chrome-devtools MCP): add newline at input max height, remove newlines, and grow below the cap all stay pinned (dist ≈ 0); a user scrolled up 400px is untouched.
bun run lintandbun run check:api-validation:strictpass.Checklist
🤖 Generated with Claude Code
https://claude.ai/code/session_01QtgAAuHzbSoS4FEYZnHdAJ