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

Make the Command Line Mode easier to access #8322

Closed
Don-Vito opened this issue Nov 18, 2020 · 6 comments
Closed

Make the Command Line Mode easier to access #8322

Don-Vito opened this issue Nov 18, 2020 · 6 comments
Labels
Area-CmdPal Command Palette issues and features Area-User Interface Issues pertaining to the user interface of the Console or Terminal Issue-Task It's a feature request, but it doesn't really need a major design. Product-Terminal The new Windows Terminal. Resolution-Fix-Committed Fix is checked in, but it might be 3-4 weeks until a release.
Milestone

Comments

@Don-Vito
Copy link
Contributor

Don-Vito commented Nov 18, 2020

Description of the new feature/enhancement

Today, to get to the command line mode I need to open the action mode and then to delete a ">" symbol, which is not very convenient.

While discussing this with @zadjii-msft few ideas were raised:

  1. Adding a separate binding for ToggleCommandPalette(mode=commandline)
  2. Having ToggleCommandPalette() switch between modes: closed->action->command line->closed.
  3. Both of them ☺️

Proposed technical implementation details (optional)

The most straightforward solution is to add an optional command palette mode to the ToggleCommandPalette, with two "launch modes": action and command line.

We will be able to implement option 2 only once #8044 gets merged.

@Don-Vito Don-Vito added the Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. label Nov 18, 2020
@ghost ghost added Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements labels Nov 18, 2020
@zadjii-msft
Copy link
Member

I'll be danged, I could have sworn I had already filed a task for this.

Ideas:

  • I don't really care that toggle with a mode might also close it. Maybe if you're in action mode, and you press the toggleCommandPalette(mode=commandline) keybinding, it'll pop you directly into commandline mode? (with Enable shortcut while CommandPalette is open #8044) (and vice versa)
  • Action and commandline mode definitely, tab search maybe? I suppose { action: toggleCommandPalette, mode: tabSearch } doesn't really make sense now does it.

@zadjii-msft zadjii-msft added Area-User Interface Issues pertaining to the user interface of the Console or Terminal Issue-Task It's a feature request, but it doesn't really need a major design. Product-Terminal The new Windows Terminal. and removed Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. labels Nov 18, 2020
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label Nov 18, 2020
@zadjii-msft zadjii-msft added this to the Terminal v2.0 milestone Nov 18, 2020
@DHowett
Copy link
Member

DHowett commented Nov 18, 2020

I'd say that the command palette being the backend for tab search is an implementation detail, not something to expose to the users 😄

@Don-Vito
Copy link
Contributor Author

Don-Vito commented Nov 18, 2020

I'll be danged, I could have sworn I had already filed a task for this.

Ideas:

* I don't really care that toggle with a mode might also close it. Maybe if you're in action mode, and you press the `toggleCommandPalette(mode=commandline)` keybinding, it'll pop you directly into commandline mode? (with #8044) (and vice versa)

* Action and commandline mode definitely, tab search maybe? I suppose `{ action: toggleCommandPalette, mode: tabSearch }` doesn't _really_ make sense now does it.

@zadjii-msft - I believe that:

  • toggleCommandPalette(mode=commandline) - should work from everywhere (even if the window is minimized and the PC is hibernated)
  • same for toggleCommandPalette(mode=commandline)
  • while toggleCommandPalette() with no mode specified should rotate through the modes (closed->action->command line).

WDYT?

@Don-Vito
Copy link
Contributor Author

I'd say that the command palette being the backend for tab search is an implementation detail, not something to expose to the users 😄

Yep.. that was a bad idea, especially since we already have a tabSeach binding...

@Don-Vito Don-Vito changed the title Allow binding a shortcut to openning CommandPalette in the Command Line Mode Make the Command Line Mode easier to access Nov 18, 2020
@DHowett DHowett removed the Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting label Nov 20, 2020
@Don-Vito
Copy link
Contributor Author

I'll be danged, I could have sworn I had already filed a task for this.

@zadjii-msft - I found it.. in the form of TODO 😆

 void TerminalPage::_HandleToggleCommandPalette(const IInspectable& /*sender*/,
                                                   const ActionEventArgs& args)
    {
        // TODO GH#6677: When we add support for commandline mode, first set the
        // mode that the command palette should be in, before making it visible.
        CommandPalette().EnableCommandPaletteMode();
        CommandPalette().Visibility(CommandPalette().Visibility() == Visibility::Visible ?
                                        Visibility::Collapsed :
                                        Visibility::Visible);
        args.Handled(true);
    }

@ghost ghost added the In-PR This issue has a related PR label Nov 24, 2020
@zadjii-msft zadjii-msft added the Area-CmdPal Command Palette issues and features label Dec 1, 2020
@ghost ghost removed the In-PR This issue has a related PR label Dec 3, 2020
@ghost ghost closed this as completed in f072eaf Dec 3, 2020
@ghost ghost added the Resolution-Fix-Committed Fix is checked in, but it might be 3-4 weeks until a release. label Dec 3, 2020
@microsoft microsoft deleted a comment Dec 3, 2020
@ghost
Copy link

ghost commented Jan 28, 2021

🎉This issue was addressed in #8382, which has now been successfully released as Windows Terminal Preview v1.6.10272.0.:tada:

Handy links:

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-CmdPal Command Palette issues and features Area-User Interface Issues pertaining to the user interface of the Console or Terminal Issue-Task It's a feature request, but it doesn't really need a major design. Product-Terminal The new Windows Terminal. Resolution-Fix-Committed Fix is checked in, but it might be 3-4 weeks until a release.
Projects
None yet
Development

No branches or pull requests

3 participants