Skip to content

settings_ui: Make open project settings action open settings UI#42669

Merged
Anthony-Eid merged 2 commits intomainfrom
project-settings-action-ui-migration
Nov 14, 2025
Merged

settings_ui: Make open project settings action open settings UI#42669
Anthony-Eid merged 2 commits intomainfrom
project-settings-action-ui-migration

Conversation

@Anthony-Eid
Copy link
Contributor

@Anthony-Eid Anthony-Eid commented Nov 13, 2025

This PR makes the OpenProjectSettings action open the settings UI in project settings mode for the first visible worktree, instead of opening the file. It also adds a OpenProjectSettingsFile action that maintains the old behavior.

Finally, this PR partially fixes a bug where the settings UI won't load project settings when the settings window is loaded before opening a project/workspace. This happens because the global app_state isn't correct in the Subscription that refreshes the available setting files to open. The bug is still present in some cases, but it's out of scope for this PR.

Release Notes:

  • settings ui: Project Settings action now opens settings UI instead of a file

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Nov 13, 2025
@maxdeviant maxdeviant changed the title settings_ui: Make open project settings action open settings ui settings_ui: Make open project settings action open settings UI Nov 13, 2025
@Anthony-Eid Anthony-Eid merged commit 3da4d3a into main Nov 14, 2025
24 checks passed
@Anthony-Eid Anthony-Eid deleted the project-settings-action-ui-migration branch November 14, 2025 01:06
11happy pushed a commit to 11happy/zed that referenced this pull request Dec 1, 2025
…industries#42669)

This PR makes the `OpenProjectSettings` action open the settings UI in
project settings mode for the first visible worktree, instead of opening
the file. It also adds a `OpenProjectSettingsFile` action that maintains
the old behavior.

Finally, this PR partially fixes a bug where the settings UI won't load
project settings when the settings window is loaded before opening a
project/workspace. This happens because the global `app_state` isn't
correct in the `Subscription` that refreshes the available setting files
to open. The bug is still present in some cases, but it's out of scope
for this PR.

Release Notes:

- settings ui: Project Settings action now opens settings UI instead of
a file
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.

1 participant