Skip to content

Add some polish to nested commands in the command palette - #7299

Merged
1 commit merged into
masterfrom
dev/migrie/f/7265-nested-cmdpal-polish
Aug 18, 2020
Merged

1 commit merged into
masterfrom
dev/migrie/f/7265-nested-cmdpal-polish

Conversation

@zadjii-msft

Copy link
Copy Markdown
Member

Summary of the Pull Request

cmdpal-nested-command-polish

  • Add a chevron for nested commands
  • Add the text of the parent command when entering a child command

References

PR Checklist

Validation Steps Performed

look at that gif

@ghost ghost added Area-UserInterface 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. labels Aug 14, 2020

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I love it. It's so freakin' clean.

String NoMatchesText { get; };
String SearchBoxText { get; };
String ControlName { get; };
String ParentCommandName { get; };

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

future: we may want to display the icon if it has one, too.

Text="{x:Bind KeyChordText, Mode=OneWay}" />
</Border>

<!-- xE70E is ChevronUp. Rotated 90 degrees, it's _ChevronRight_ -->

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

HA, clever

@zadjii-msft Mike Griese (zadjii-msft) added the AutoMerge Marked for automatic merge by the bot when requirements are met label Aug 18, 2020
@ghost

Copy link
Copy Markdown

Hello Mike Griese (@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 3d64921 into master Aug 18, 2020
@ghost
ghost deleted the dev/migrie/f/7265-nested-cmdpal-polish branch August 18, 2020 19:32
@ghost

Copy link
Copy Markdown

🎉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. Needs-Second It's a PR that needs another sign-off Product-Terminal The new Windows Terminal.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

A nested cmdpal menu should show the parent command above the list of commands

3 participants