Skip to content

Scroll to bottom when reloading ACP agent threads#52843

Merged
eholk merged 1 commit intomainfrom
teal-moss
Mar 31, 2026
Merged

Scroll to bottom when reloading ACP agent threads#52843
eholk merged 1 commit intomainfrom
teal-moss

Conversation

@eholk
Copy link
Copy Markdown
Contributor

@eholk eholk commented Mar 31, 2026

When reopening an ACP agent thread (e.g. after restarting Zed) that has no saved scroll position, the conversation now starts scrolled to the bottom so the user sees the latest messages instead of starting at the top.

Uses set_follow_tail(true) rather than a one-shot scroll_to_end() so the scroll position is re-anchored on every layout pass — ensuring it lands at the true bottom even when list items haven't been measured yet. The follow-tail mode automatically disengages as soon as the user scrolls up.

Release Notes:

  • Fixed agent threads starting scrolled to the top when reopened after restarting Zed.

When reopening an ACP agent thread (e.g. after restarting Zed) that has
no saved scroll position, scroll to the bottom of the conversation so
the user sees the latest messages instead of starting at the top.
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Mar 31, 2026
@zed-community-bot zed-community-bot bot added the staff Pull requests authored by a current member of Zed staff label Mar 31, 2026
@eholk eholk requested a review from benbrandt March 31, 2026 17:33
Copy link
Copy Markdown
Member

@benbrandt benbrandt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@eholk eholk merged commit bf79e32 into main Mar 31, 2026
41 checks passed
@eholk eholk deleted the teal-moss branch March 31, 2026 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants