Skip to content

Add support for LightDismissing the renamer#9733

Merged
1 commit merged intomainfrom
dev/migrie/9718-light-dismiss-renamer
Apr 7, 2021
Merged

Add support for LightDismissing the renamer#9733
1 commit merged intomainfrom
dev/migrie/9718-light-dismiss-renamer

Conversation

@zadjii-msft
Copy link
Member

@zadjii-msft zadjii-msft commented Apr 6, 2021

Summary of the Pull Request

Huh, I guess I missed making the window renamer light-dismissable. This is a oneline fix for that.

Light dismissing is treated as a cancel, not as a commit.

References

PR Checklist

Validation Steps Performed

This feels right.

@ghost ghost added Area-UserInterface Issues pertaining to the user interface of the Console or Terminal Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-3 A description (P3) Product-Terminal The new Windows Terminal. labels Apr 6, 2021
@DHowett
Copy link
Member

DHowett commented Apr 6, 2021

@msftbot merge this in 2 minutes

@ghost ghost added the AutoMerge Marked for automatic merge by the bot when requirements are met label Apr 6, 2021
@ghost
Copy link

ghost commented Apr 6, 2021

Hello @DHowett!

Because you've given me some instructions on how to help merge this pull request, I'll be modifying my merge approach. Here's how I understand your requirements for merging this pull request:

  • I won't merge this pull request until after the UTC date Tue, 06 Apr 2021 21:31:40 GMT, which is in 2 minutes

If this doesn't seem right to you, you can tell me to cancel these instructions and use the auto-merge policy that has been configured for this repository. Try telling me "forget everything I just told you".

@DHowett
Copy link
Member

DHowett commented Apr 6, 2021

ah, a conflict!

@DHowett
Copy link
Member

DHowett commented Apr 6, 2021

wait, what? something was conflicted..

@ghost ghost merged commit cdf2630 into main Apr 7, 2021
@ghost ghost deleted the dev/migrie/9718-light-dismiss-renamer branch April 7, 2021 00:12
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-Bug It either shouldn't be doing this or needs an investigation. Priority-3 A description (P3) Product-Terminal The new Windows Terminal.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BASH] Window renamer doesn't dismiss when you click outside of it

3 participants