Skip to content

project_panel: Always open panel on pane's reveal in project panel#53539

Merged
dinocosta merged 1 commit intomainfrom
23967-refactor
Apr 13, 2026
Merged

project_panel: Always open panel on pane's reveal in project panel#53539
dinocosta merged 1 commit intomainfrom
23967-refactor

Conversation

@dinocosta
Copy link
Copy Markdown
Member

Update the way pane::RevealInProjectPanel is handled to ensure that, regardless of whether the file belongs to any open project, the project panel is always activated and focused.

This refactor is a result of some internal feedback after changing its handling so as to show a notification stating that the item that the user was trying to reveal didn't belong to an open project – #51246 .

We feel users are probably already used to relying on cmd-shift-e (on macOS), in pretty much every context, in order to open the project panel, and so having situations where it doesn't actually happen seems like a bad user experience.

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

Relates to #23967

Release Notes:

  • Improved pane: reveal in project panel to open the project panel, even if working with an unsaved buffer.
Update the way `pane::RevealInProjectPanel` is handled to ensure that,
regardless of whether the file belongs to any open project, the project
panel is always activated and focused.

This refactor is a result of some internal feedback after changing its
handling so as to show a notification stating that the item that the
user was trying to reveal didn't belong to an open project.

We feel users are probably already used to relying on `cmd-shift-e` (on
macOS), in pretty much every context, in order to open the project
panel, and so having situations where it doesn't actually happen seems
like a bad user experience.
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Apr 9, 2026
@dinocosta dinocosta requested a review from maxbrunsfeld April 9, 2026 16:10
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Apr 9, 2026
@dinocosta dinocosta changed the title refactor: always open project panel on pane's reveal in project panel Apr 9, 2026
@dinocosta dinocosta merged commit 25e02cb into main Apr 13, 2026
43 checks passed
@dinocosta dinocosta deleted the 23967-refactor branch April 13, 2026 20:47
@injust
Copy link
Copy Markdown
Contributor

injust commented Apr 16, 2026

Does this need to be cherry-picked to stable? The stable release notes mention this PR, but it isn't part of the release.

Ideally, this should've landed in the same release as #51246.

@dinocosta
Copy link
Copy Markdown
Member Author

Does this need to be cherry-picked to stable? The stable release notes mention this PR, but it isn't part of the release.

Ideally, this should've landed in the same release as #51246.

Definitely! Thank you for calling that out, I had a note to cherry pick to preview right after merging but somehow forgot to do that 🤦

Will coordinate with the team to see if it makes sense to do a patch release including this, and likely other changes. Thanks!

@rtfeldman
Copy link
Copy Markdown
Contributor

/cherry-pick stable

@rtfeldman
Copy link
Copy Markdown
Contributor

/cherry-pick preview

github-actions Bot pushed a commit that referenced this pull request Apr 16, 2026
…53539)

Update the way `pane::RevealInProjectPanel` is handled to ensure that,
regardless of whether the file belongs to any open project, the project
panel is always activated and focused.

This refactor is a result of some internal feedback after changing its
handling so as to show a notification stating that the item that the
user was trying to reveal didn't belong to an open project
– #51246 .

We feel users are probably already used to relying on `cmd-shift-e` (on
macOS), in pretty much every context, in order to open the project
panel, and so having situations where it doesn't actually happen seems
like a bad user experience.

Relates to #23967 

Release Notes:

- Improved `pane: reveal in project panel` to open the project panel,
even if working with an unsaved buffer.
@zed-zippy
Copy link
Copy Markdown
Contributor

zed-zippy Bot commented Apr 16, 2026

@eholk eholk mentioned this pull request Apr 16, 2026
18 tasks
eholk added a commit that referenced this pull request Apr 16, 2026
We're doing another preview release today to get our latest parallel
agents features out there. This PR is a rollup of all the changes we
need to cherry pick.

- [x] 7a26e48 from
#54081
- [x] #54066
- [x] #53992
- [x] #54079
- [x] #54057
- [x] #54056
- [x] #54052
- [x] #53999
- [x] #54009
- [x] #53539 (already cherry
picked as 25e02cb)
- [x] #54070
- [x] #54053
- fix `run_tests.yml`: 67e92b5
- [x] #53979
- [x] #53884
- [x] #54067
- [x] #54014 (already on
branch)
- [x] #54030 (already on
branch)
- [x] #54094

Release Notes:

- N/A

---------

Co-authored-by: Lukas Wirth <me@lukaswirth.dev>
Co-authored-by: Bennet Bo Fenner <bennet@zed.dev>
Co-authored-by: Nathan Sobo <nathan@zed.dev>
Co-authored-by: Ben Brandt <benjamin.j.brandt@gmail.com>
Co-authored-by: Bennet Bo Fenner <bennetbo@gmx.de>
Co-authored-by: Danilo Leal <67129314+danilo-leal@users.noreply.github.com>
Co-authored-by: Neel <neel@zed.dev>
Co-authored-by: Richard Feldman <richard@zed.dev>
piper-of-dawn pushed a commit to piper-of-dawn/zed that referenced this pull request Apr 25, 2026
…ed-industries#53539)

Update the way `pane::RevealInProjectPanel` is handled to ensure that,
regardless of whether the file belongs to any open project, the project
panel is always activated and focused.

This refactor is a result of some internal feedback after changing its
handling so as to show a notification stating that the item that the
user was trying to reveal didn't belong to an open project
– zed-industries#51246 .

We feel users are probably already used to relying on `cmd-shift-e` (on
macOS), in pretty much every context, in order to open the project
panel, and so having situations where it doesn't actually happen seems
like a bad user experience.

Relates to zed-industries#23967 

Release Notes:

- Improved `pane: reveal in project panel` to open the project panel,
even if working with an unsaved buffer.
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

4 participants