-
Notifications
You must be signed in to change notification settings - Fork 8
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
Command Palette integration #64
Comments
Thanks for the suggestion! Code actions (as shown in the code action menu) are context specific and are not available most of the time. Adding them to the command palette would add a lot of commands which won't be working most of the time. I'm keeping this issue open, in case I find a way to filter/hide them when they are not available for the given cursor location. |
Good point, I'm surprised that there's not a way to make them context-dependent, doesn't look like there is... I wonder how big of an issue it is to have not-working commands available, however? I guess there are three reasons I can think of to avoid them, and then my thoughts on those...
|
I've never used the Refactor menu in VSCode, so I was surprised to find out it doesn't seem to be integrated with the Command Palette (I can't find Refactor commands through it, like Surround With etc.).
The Command Palette is my preferred method of interaction with this type of command so I don't have to remember custom keyboard shortcuts (or god forbid, use a mouse 😄 ).
Is there a way to add some/all of these commands to the Command Palette? Or is there a design reason those aren't tied together?
Thanks! Love the power of the extension, just trying to figure out how to fit it in my workflow.
The text was updated successfully, but these errors were encountered: