Skip to content

Preserve terminal spinners in thread titles - #57983

Merged
rtfeldman merged 1 commit into
mainfrom
AI-304/preserve-terminal-spinners
May 29, 2026
Merged

Preserve terminal spinners in thread titles#57983
rtfeldman merged 1 commit into
mainfrom
AI-304/preserve-terminal-spinners

Conversation

@rtfeldman

Copy link
Copy Markdown
Contributor

Summary:

  • Preserve spinner/logo prefixes from live terminal titles when terminal threads have custom titles.
  • Store raw terminal titles and custom user titles separately, recomposing display titles on demand.
  • Keep spinner prefixes out of the title editor while preserving sidebar/search display behavior.

Tests:

  • cargo test -p agent_ui test_terminal_custom_title_recomposes_with_live_spinner -- --nocapture
  • cargo test -p agent_ui test_terminal_title_editor_excludes_spinner_prefix -- --nocapture
  • cargo test -p sidebar test_agent_panel_terminals_appear_in_sidebar_and_search -- --nocapture

Closes AI-304

Release Notes:

  • Fixed terminal thread titles to preserve animated spinner and logo prefixes after renaming.

@rtfeldman rtfeldman self-assigned this May 28, 2026
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label May 28, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label May 28, 2026
@rtfeldman
rtfeldman marked this pull request as ready for review May 29, 2026 03:45
@rtfeldman
rtfeldman requested a review from benbrandt May 29, 2026 03:45
@rtfeldman
rtfeldman added this pull request to the merge queue May 29, 2026
Merged via the queue into main with commit e5f5767 May 29, 2026
45 checks passed
@rtfeldman
rtfeldman deleted the AI-304/preserve-terminal-spinners branch May 29, 2026 03:49
@zed-zippy zed-zippy Bot added the PR state:needs review Used to label PRs that are in need of a post-merge approval label May 29, 2026
@benbrandt

Copy link
Copy Markdown
Member

@zed-industries/approved

@zed-zippy zed-zippy Bot removed the PR state:needs review Used to label PRs that are in need of a post-merge approval label Jun 1, 2026
TomPlanche pushed a commit to TomPlanche/zed that referenced this pull request Jun 2, 2026
Summary:

- Preserve spinner/logo prefixes from live terminal titles when terminal
threads have custom titles.
- Store raw terminal titles and custom user titles separately,
recomposing display titles on demand.
- Keep spinner prefixes out of the title editor while preserving
sidebar/search display behavior.

Tests:

- cargo test -p agent_ui
test_terminal_custom_title_recomposes_with_live_spinner -- --nocapture
- cargo test -p agent_ui
test_terminal_title_editor_excludes_spinner_prefix -- --nocapture
- cargo test -p sidebar
test_agent_panel_terminals_appear_in_sidebar_and_search -- --nocapture

Closes AI-304

Release Notes:

- Fixed terminal thread titles to preserve animated spinner and logo
prefixes after renaming.
This was referenced Jun 18, 2026
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
Summary:

- Preserve spinner/logo prefixes from live terminal titles when terminal
threads have custom titles.
- Store raw terminal titles and custom user titles separately,
recomposing display titles on demand.
- Keep spinner prefixes out of the title editor while preserving
sidebar/search display behavior.

Tests:

- cargo test -p agent_ui
test_terminal_custom_title_recomposes_with_live_spinner -- --nocapture
- cargo test -p agent_ui
test_terminal_title_editor_excludes_spinner_prefix -- --nocapture
- cargo test -p sidebar
test_agent_panel_terminals_appear_in_sidebar_and_search -- --nocapture

Closes AI-304

Release Notes:

- Fixed terminal thread titles to preserve animated spinner and logo
prefixes after renaming.
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