Skip to content

Add support for Commandline Mode to the CmdPal#7293

Merged
92 commits merged intomasterfrom
dev/migrie/f/cmdpal-commandline-mode
Aug 24, 2020
Merged

Add support for Commandline Mode to the CmdPal#7293
92 commits merged intomasterfrom
dev/migrie/f/cmdpal-commandline-mode

Conversation

@zadjii-msft
Copy link
Member

Summary of the Pull Request

Adds support for "commandline mode" to the command palette.
cmdpal-commandline-mode

This allows the user to start typing a wt.exe commandline directly in the command palette, to run that commandline directly in the current window. This allows the user input something like > nt -p Ubuntu ; sp -p ssh and open up a new tab and split it in the current window.

References

PR Checklist

Validation Steps Performed

Tested manually

…ommand-Palette-v2

# Conflicts:
#	src/cascadia/TerminalApp/TerminalPage.h
  Don't die when we encounter an unexpected key
  Reload successfully
…ommand-Palette-v2

# Conflicts:
#	src/cascadia/TerminalApp/GlobalAppSettings.cpp
#	src/cascadia/TerminalApp/GlobalAppSettings.h
# Conflicts:
#	src/cascadia/TerminalApp/ActionAndArgs.cpp
#	src/cascadia/TerminalApp/AppKeyBindingsSerialization.cpp
@DHowett
Copy link
Member

DHowett commented Aug 14, 2020

The.. people who write the typescript package that integrates C++ with VS Code might .. be able to help us with .. commandline parsing? 🤷

@carlos-zamora carlos-zamora removed their assignment Aug 19, 2020
@carlos-zamora carlos-zamora removed their assignment Aug 19, 2020
Copy link
Contributor

@leonMSFT leonMSFT left a comment

Choose a reason for hiding this comment

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

Just a couple of spots were missed in the keyhandler that need to include the tab action modes, but damn it's so cool how typing one char changes everything 😎

@ghost ghost added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Aug 21, 2020
@leonMSFT leonMSFT removed their assignment Aug 21, 2020
@ghost ghost removed the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Aug 24, 2020
@zadjii-msft
Copy link
Member Author

This PR is currently blocked on the CI issue fixed over in #7383

@msftbot merge this in 5 minutes

@zadjii-msft zadjii-msft added the AutoMerge Marked for automatic merge by the bot when requirements are met label Aug 24, 2020
@ghost
Copy link

ghost commented Aug 24, 2020

Hello @zadjii-msft!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit f897ce0 into master Aug 24, 2020
@ghost ghost deleted the dev/migrie/f/cmdpal-commandline-mode branch August 24, 2020 21:39
@ghost
Copy link

ghost commented Aug 26, 2020

🎉Windows Terminal Preview v1.3.2382.0 has been released which incorporates this pull request.:tada:

Handy links:

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-UserInterface Issues pertaining to the user interface of the Console or Terminal AutoMerge Marked for automatic merge by the bot when requirements are met Issue-Task It's a feature request, but it doesn't really need a major design. Priority-1 A description (P1) Product-Terminal The new Windows Terminal.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Command Palette: Add support for commandline mode

7 participants