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

#1536 - Feature Request - Duplicate tab #1685

Merged
merged 9 commits into from
Jul 2, 2019
Merged

#1536 - Feature Request - Duplicate tab #1685

merged 9 commits into from
Jul 2, 2019

Conversation

dnagl
Copy link
Contributor

@dnagl dnagl commented Jun 27, 2019

Added keybinding option to duplicate focused tab

Summary of the Pull Request

Added necessary delcarations and implmentations to provide a new keybinding for duplicating the focused tab.

References

PR Checklist

Detailed Description of the Pull Request / Additional comments

As discussed in this issue, a simple implementation for duplicating the focused tab was made by me.
I added the needed declarations for the new keybinding and added a method to duplicate the focused tab in App.cpp.

Validation Steps Performed

  • Add keybinding element to profiles.json
    • "command" : "duplicateTab"
    • "keys": [ "ctrl+shift+d" ]
  • Open Terminal
  • Verify other features and functionalities work as usual without unexpected behaviour
  • Use configured keybinding to validate the duplication of the focused tab

@carlos-zamora carlos-zamora self-requested a review June 27, 2019 22:30
Copy link
Member

@carlos-zamora carlos-zamora left a comment

Choose a reason for hiding this comment

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

Please run the Code Formatter located in tools\OpenConsole.psm1

Other than that, pretty straightforward. Thanks!

src/cascadia/TerminalApp/App.cpp Outdated Show resolved Hide resolved
@ghost ghost added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Jun 27, 2019
@JushBJJ
Copy link

JushBJJ commented Jun 28, 2019

Awesome feature that is needed!

@ghost ghost removed the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Jun 28, 2019
@dnagl
Copy link
Contributor Author

dnagl commented Jun 28, 2019

@JushBJJ I also think that this is a great feature in this project. This is one of multiple steps. in #1535 there was a discussion about three different stages of this feature which I hope we can see them in the first release.

Copy link
Member

@miniksa miniksa left a comment

Choose a reason for hiding this comment

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

fix tabs/spacing issues and I think its good

@ghost ghost added Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Jun 28, 2019
@dnagl
Copy link
Contributor Author

dnagl commented Jun 29, 2019

@miniksa You were right. I did not set this option in Visual Studio in the first place. The changes are made.
Thanks for the review ;)

Copy link
Member

@zadjii-msft zadjii-msft left a comment

Choose a reason for hiding this comment

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

Looks solid to me.

Copy link
Member

@miniksa miniksa left a comment

Choose a reason for hiding this comment

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

I'm good with this, but I'm going to try to see why the line Carlos marked is a diff despite looking exactly the same before I hit the merge button.

@ghost
Copy link

ghost commented Aug 3, 2019

🎉Windows Terminal Preview v0.3.2142.0 has been released which incorporates this pull request.:tada:

Handy links:

@ghost ghost mentioned this pull request Aug 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature Request - Duplicate tab (Basic)
5 participants