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

Add Selector run and clear commands #33

Merged

Conversation

srchase
Copy link
Contributor

@srchase srchase commented May 16, 2022

This PR adds two selector commands which use the selector protocol extension added to the Smithy Language Server in smithy-lang/smithy-language-server#36.

From the command palette, two commands are now available, Smithy:Selector:Run and Smithy:Selector:Clear:

smithy_commands

When invoked, the Smithy:Selector:Run command opens a dialog box where a selector expression can be entered to query the Smithy model loaded in the workspace:

expresion_input

Pressing enter runs the selector expression against the model, highlighting any shapes that match the query:

highlighted

If the command is invoked a again, the previously evaluated expression is pre-populated in the input box and the prior results are cleared.

The Smithy:Selector:Clear command clears previously highlighted results.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@srchase srchase requested a review from a team as a code owner May 16, 2022 20:43
@srchase srchase merged commit 35f402e into smithy-lang:lang-server-integration May 17, 2022
@srchase srchase deleted the find-by-selector branch May 17, 2022 16:14
@srchase srchase mentioned this pull request May 26, 2022
srchase pushed a commit to srchase/smithy-vscode that referenced this pull request Jun 9, 2022
srchase pushed a commit that referenced this pull request Jun 9, 2022
srchase pushed a commit that referenced this pull request Jun 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants