Open axel7083 opened 4 months ago
My suggestion would be to remove the CommandPalette.svelte
and make the logic in the main
package, and uses the QuickPickInput.svelte
component instead for the commands. The CommandRegistry
should probably be able to handle that easily.
CommandRegistry
First I would add the InputQuickPickRegistry
in the CommandRegistry class constructor.
I would add a void showCommandPalette()
responsible of showing a quick pick with the commands.
cc @benoitf what do you think ? I am currently working on the modals so I can make the change
@axel7083 I would not do it now, I think there is already an issue about merging CommandPalette into another component, etc.
Command Palette is still currently quite hidden, so I would prefer to focus on the light theme even and probably this one would be done at the very last.
Is your enhancement related to a problem? Please describe
Describe the solution you'd like
Describe alternatives you've considered
No response
Additional context
No response