Skip to content

thread_view: Fix the thinking toggle button - #59111

Merged
danilo-leal merged 1 commit into
mainfrom
thinking-toggle-fix
Jun 11, 2026
Merged

thread_view: Fix the thinking toggle button#59111
danilo-leal merged 1 commit into
mainfrom
thinking-toggle-fix

Conversation

@danilo-leal

Copy link
Copy Markdown
Member

We recently changed this given some new models (e.g., Fable) don't allow turning thinking off. In those cases, we are not rendering the split button anymore, but we should still display the thinking icon, so they look a bit more consistent.

Release Notes:

  • N/A

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Jun 11, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Jun 11, 2026
@danilo-leal
danilo-leal added this pull request to the merge queue Jun 11, 2026
Merged via the queue into main with commit 8432a26 Jun 11, 2026
43 checks passed
@danilo-leal
danilo-leal deleted the thinking-toggle-fix branch June 11, 2026 14:20
This was referenced Jun 18, 2026
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
We recently changed this given some new models (e.g., Fable) don't allow
turning thinking off. In those cases, we are not rendering the split
button anymore, but we should still display the thinking icon, so they
look a bit more consistent.

Release Notes:

- N/A
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
We recently changed this given some new models (e.g., Fable) don't allow
turning thinking off. In those cases, we are not rendering the split
button anymore, but we should still display the thinking icon, so they
look a bit more consistent.

Release Notes:

- N/A
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