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

Preferences: Additional Open Preferences Commands #9785

Merged
merged 1 commit into from
Jul 28, 2021

Conversation

colin-grant-work
Copy link
Contributor

@colin-grant-work colin-grant-work commented Jul 27, 2021

This is a rebased and modified version of #9410.

Fixes #7328

  • Adds Preferences: Open User Preferences command, which opens the preferences-view with User scope.
  • Adds Preferences: Open Workspace Preferences command, which opens the preferences-view with Workspace scope.
  • Modifies Settings: Open Preferences command to be able to accept an optional query parameter. Pre-populates the search with this query upon opening the preferences-view.

How to test

  1. Open the command palette and search Preferences: Open User Preferences or Preferences: Open Workspace Preferences.
  2. Execute either command and confirm that the preferences-view is opened with the correct scope.
  3. Use the .vsix provided here, to pass queries to the Settings: Open Preferences command. Observe that the preferences-view opens with the correct search results displayed.

Review checklist

Reminder for reviewers

Signed-off-by: seantan22 [email protected]
Signed-off-by: Colin Grant [email protected]

What it does
- Adds `Preferences: Open User Preferences` command, which opens the
  _preferences-view_ with `User` scope.
- Adds `Preferences: Open Workspace Preferences` command, which opens
  the _preferences-view_ with `Workspace` scope.
- Modifies `Settings: Open Preferences` command to be able to accept an
  optional `query` parameter. Pre-populates the search with this query
upon opening the _preferences-view_.

How to test
1. Open the command palette and search `Preferences: Open User
Preferences` or `Preferences: Open Workspace Preferences`.
2. Execute either command and confirm that the _preferences-view_ is
opened with the correct scope.

Signed-off-by: seantan22 <[email protected]>
Signed-off-by: Colin Grant <[email protected]>

Co-authored-by: seantan22 <[email protected]>
Co-authored-by: Colin Grant <[email protected]>
@vince-fugnitto vince-fugnitto added the preferences issues related to preferences label Jul 27, 2021
Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

The changes look good and work well for the different use-cases 👍

Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

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

Confirmed that everything works well:

  • Opening settings (default)
  • Opening user preferences
  • Opening workspace preferences
  • Opening workspace preferences while in user preferences switches the scope correctly
  • Using the extension to open preferences with/without a query.

@colin-grant-work colin-grant-work merged commit 3716c07 into master Jul 28, 2021
@colin-grant-work colin-grant-work deleted the feature/additional-pref-commands branch July 28, 2021 14:05
@github-actions github-actions bot added this to the 1.16.0 milestone Jul 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preferences issues related to preferences
Projects
None yet
Development

Successfully merging this pull request may close these issues.

preferences: the command 'open settings' should support a 'query'
3 participants