Skip to content

Change the default panel layout#52232

Merged
mikayla-maki merged 5 commits intomainfrom
agent-panel-layout-migration
Mar 23, 2026
Merged

Change the default panel layout#52232
mikayla-maki merged 5 commits intomainfrom
agent-panel-layout-migration

Conversation

@mikayla-maki
Copy link
Copy Markdown
Member

@mikayla-maki mikayla-maki commented Mar 23, 2026

This PR adjusts the default panel layout for anyone on the agent v2 feature flag.

Note that this changes the right status bar items to show in reverse priority order, and then adjusts priorities so that the "project management" buttons appear to the right, and the outline panel appears to the far left. The reversal "cancels out" most of the priority changes, except the outline panel and the collab panel.

Self-Review Checklist

  • I've reviewed my own diff for quality, security, and reliability
  • Unsafe blocks (if any) have justifying comments
  • The content is consistent with the UI/UX checklist
  • Tests cover the new/changed behavior
  • Performance impact has been considered and is acceptable

Release Notes:

  • Swapped the order of the collab and outline status bar buttons

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Mar 23, 2026
@zed-community-bot zed-community-bot bot added the staff Pull requests authored by a current member of Zed staff label Mar 23, 2026
Change the default dock positions so the agent panel defaults to the
left (near the editor) and project/outline/git/collab panels default
to the right.

For existing users, a one-time migration pins their current panel
positions into settings.json so the new defaults don't rearrange
their layout. A notification informs them of the change.
@mikayla-maki mikayla-maki force-pushed the agent-panel-layout-migration branch from 55e41f2 to dcaeb19 Compare March 23, 2026 20:09
@github-actions github-actions bot added Size S and removed Size M labels Mar 23, 2026
@mikayla-maki mikayla-maki marked this pull request as ready for review March 23, 2026 20:15
@zed-codeowner-coordinator zed-codeowner-coordinator bot requested review from a team, benbrandt and bennetbo and removed request for a team March 23, 2026 20:15
@mikayla-maki mikayla-maki enabled auto-merge (squash) March 23, 2026 20:23
@mikayla-maki mikayla-maki disabled auto-merge March 23, 2026 20:23
@mikayla-maki mikayla-maki enabled auto-merge (squash) March 23, 2026 20:24
@mikayla-maki mikayla-maki merged commit 78df135 into main Mar 23, 2026
31 checks passed
@mikayla-maki mikayla-maki deleted the agent-panel-layout-migration branch March 23, 2026 20:51
AmaanBilwar pushed a commit to AmaanBilwar/zed that referenced this pull request Mar 23, 2026
This PR adjusts the default panel layout for anyone on the agent v2
feature flag.

Note that this changes the right status bar items to show in reverse
priority order, and then adjusts priorities so that the "project
management" buttons appear to the right, and the outline panel appears
to the far left. The reversal "cancels out" most of the priority
changes, except the outline panel and the collab panel.


## Self-Review Checklist

<!-- Check before requesting review: -->
- [x] I've reviewed my own diff for quality, security, and reliability
- [x] Unsafe blocks (if any) have justifying comments
- [x] The content is consistent with the [UI/UX
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
- [x] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable

Release Notes:

- Swapped the order of the collab and outline status bar buttons
maxbrunsfeld added a commit that referenced this pull request Mar 25, 2026
## Context

Since #52232, there have been
a lot of problems related to panels randomly hiding and showing
themselves. However, the code in that PR is pretty innocuous. It
shouldn't be having that kind of effect. However, it turns out there's a
bug in how GPUI subscriptions work that caused there to be orphaned
observe callbacks firing spuriously. The core sequence is captured in
`test_unsubscribe_during_callback_with_insert`, essentially you have to
have two observers on the same item (`SettingsStore`, in this case),
that both drop themselves. If the first observer _adds_ a callback onto
the _same item_, then the second observer will never successfully drop
itself.

While in there, I also fixed an unrelated bug that @maxbrunsfeld noticed
where if you have two callbacks on the same item, and an earlier
callback drops a later one, then the second callback will spuriously
fire.

I also added a few extra smoke tests to the subscription code, and a
test capturing the observed bug at the workspace level.

## Self-Review Checklist

<!-- Check before requesting review: -->
- [x] I've reviewed my own diff for quality, security, and reliability
- [x] Unsafe blocks (if any) have justifying comments
- [x] The content is consistent with the [UI/UX
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
- [x] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable

Release Notes:

- N/A

---------

Co-authored-by: max <[email protected]>
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 Size S staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants