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

Store starts open state of outline panel #13601

Merged
merged 1 commit into from
Jun 27, 2024

Conversation

yongkangchen
Copy link
Contributor

Release Notes:

  • Fixed issue where outline panel remains open despite being closed before window close.

Before the release of Outline Panel feature, everything works fine. But after that, the outline panel keeps open. It's very annoy that I only want to edit a simple file. Event I close it before I close the window. The active state of this panel didn't stored.

Description:

Before the introduction of the Outline Panel feature, the application behaved as expected. However, with the addition of the Outline Panel, an issue arose where the panel would persistently remain open. This behavior was observed even when manually closing the panel before closing the application window. The problem stemmed from the inactive state of the panel not being stored properly. This fix addresses the issue by ensuring that the panel's active state is correctly stored and retrieved, thereby improving user experience and preventing unnecessary persistence of the panel's visibility.

Screen Records

Before Release of Outline Panel

before_issue.mov

After Release of Outline Panel

before.mov

After Fixing This Issue

fixed_1080p.mov

Copy link

cla-bot bot commented Jun 27, 2024

We require contributors to sign our Contributor License Agreement, and we don't have @yongkangchen 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'.

@yongkangchen
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 27, 2024
Copy link

cla-bot bot commented Jun 27, 2024

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

Copy link
Contributor

@SomeoneToIgnore SomeoneToIgnore left a comment

Choose a reason for hiding this comment

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

Thank you.

Curious: does project panel bother you similarly? Seems that it does not have this code and manages to work correctly?

@SomeoneToIgnore SomeoneToIgnore self-assigned this Jun 27, 2024
@yongkangchen
Copy link
Contributor Author

Thank you.

Curious: does project panel bother you similarly? Seems that it does not have this code and manages to work correctly?

No, because the project panel only opens when there is a project opened.

@SomeoneToIgnore SomeoneToIgnore merged commit 228202a into zed-industries:main Jun 27, 2024
9 checks passed
SomeoneToIgnore pushed a commit that referenced this pull request Jul 1, 2024
- Fixed issue where outline panel remains open despite being closed
before window close.

Before the release of Outline Panel feature, everything works fine. But
after that, the outline panel keeps open. It's very annoy that I only
want to edit a simple file. Event I close it before I close the window.
The active state of this panel didn't stored.

### Description:
Before the introduction of the Outline Panel feature, the application
behaved as expected. However, with the addition of the Outline Panel, an
issue arose where the panel would persistently remain open. This
behavior was observed even when manually closing the panel before
closing the application window. The problem stemmed from the inactive
state of the panel not being stored properly. This fix addresses the
issue by ensuring that the panel's active state is correctly stored and
retrieved, thereby improving user experience and preventing unnecessary
persistence of the panel's visibility.

### Screen Records

#### Before Release of Outline Panel


https://github.com/zed-industries/zed/assets/704762/2a222c70-c6d7-4472-9f27-7868d1786a5f


#### After Release of Outline Panel


https://github.com/zed-industries/zed/assets/704762/69c16a5d-beed-4d4a-8341-83c53f6a6713


#### After Fixing This Issue


https://github.com/zed-industries/zed/assets/704762/f51c5df7-54e3-4a62-ac54-b5d12cfe69d1

### Release Notes:

- Persist outline panel open state to avoid opening it on Zed startup
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.

2 participants