Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent folder expansion when all items are closed #12729

Merged
merged 2 commits into from
Jun 6, 2024

Conversation

panghu-huang
Copy link
Contributor

Release Notes:

  • Prevent folder expansion when all items are closed

Problem

When all items are closed, the next activated file expands (see the video below).

2024-06-06.20.42.48.mov

Cause

When the currently active item is closed, Zed tries to activate the previously active item. Activating an item by default expands the corresponding folder, which can result in folders being expanded when all files are closed.

Fixed Video

2024-06-06.20.46.02.mov

Copy link

cla-bot bot commented Jun 6, 2024

We require contributors to sign our Contributor License Agreement, and we don't have @panghu-huang on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@panghu-huang
Copy link
Contributor Author

@cla-bot check

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Jun 6, 2024
Copy link

cla-bot bot commented Jun 6, 2024

The cla-bot has been summoned, and re-checked this pull request!

Copy link
Contributor

@osiewicz osiewicz left a comment

Choose a reason for hiding this comment

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

LGTM

@osiewicz osiewicz merged commit fd39f20 into zed-industries:main Jun 6, 2024
8 checks passed
@panghu-huang panghu-huang deleted the fix-close-all-items branch June 7, 2024 07:22
ming900518 pushed a commit to ming900518/zed that referenced this pull request Jun 14, 2024
)

Release Notes:

- Prevent folder expansion when all items are closed

### Problem
When all items are closed, the next activated file expands (see the
video below).


https://github.com/zed-industries/zed/assets/21101490/a7631cd2-4e97-4954-8b01-d283dd4796be


### Cause
When the currently active item is closed, Zed tries to activate the
previously active item. Activating an item by default expands the
corresponding folder, which can result in folders being expanded when
all files are closed.

### Fixed Video


https://github.com/zed-industries/zed/assets/21101490/d30f05c5-6d86-4e11-b349-337fa75586f3
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants