-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Duplicated tabs open next to the current tab #14521
Conversation
@microsoft-github-policy-service agree |
Thanks for the contribution! Sorry for the delay in reviewing it. As we enter the holidays, we're going to be running a bit short-staffed. 😄 |
@zadjii-msft I am 100% okay saying "duplicated tabs open next to the current tab" actually... |
Hello @zadjii-msft! Because this pull request has the 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 (
|
🎉 Handy links: |
Hi, I know this issue is closed, but I actually find it annoying. I liked that duplicate tabs would open at the end... not next to the one that is duplicated. Can you please provide setting, where this can be changed between two options? |
Summary of the Pull Request
Added an additional optional parameter that indicates the position of the new tab being created.
TerminalDuplicateTabIssue_BeforeFix.mp4
TerminalDuplicateTabIssue_AfterFix.mp4
PR Checklist
Detailed Description of the Pull Request / Additional comments
An optional parameter is added to TerminalPage::_CreateNewTabPane() and TerminalPage::_InitializeTab() which indicates the insert position of the duplicated tab.
During a new tab creation (not duplicate), this optional parameter has a default value(-1) and the new tab is inserted at the end.
The duplicated tab is inserted next to the original one even if it is not focused.
Validation Steps Performed
Created different tabs and duplicated them.
Duplicated tabs that are focused and not focused.