Skip to content

Simplify TerminalPage by introducing _GetFocusedTabImpl#8655

Merged
1 commit merged intomicrosoft:mainfrom
Don-Vito:8415-focused-tab-impl
Jan 4, 2021
Merged

Simplify TerminalPage by introducing _GetFocusedTabImpl#8655
1 commit merged intomicrosoft:mainfrom
Don-Vito:8415-focused-tab-impl

Conversation

@Don-Vito
Copy link
Contributor

A part of the #8415.
Very technical commit to simplify the terminal page code
towards additional steps of simplifying tab management.
No business logic should change.

A firs step in splitting #8427

@zadjii-msft zadjii-msft self-assigned this Jan 4, 2021
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.

T H A N K Y O U

@zadjii-msft zadjii-msft removed their assignment Jan 4, 2021
@zadjii-msft zadjii-msft added Area-CodeHealth Issues related to code cleanliness, linting, rules, warnings, errors, static analysis, etc. Product-Terminal The new Windows Terminal. labels Jan 4, 2021
Copy link
Member

@DHowett DHowett left a comment

Choose a reason for hiding this comment

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

Love it so much. Happy new year 😄

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

ghost commented Jan 4, 2021

Hello @DHowett!

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 9b07cb8 into microsoft:main Jan 4, 2021
@ghost
Copy link

ghost commented Jan 28, 2021

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

Handy links:

mpela81 pushed a commit to mpela81/terminal that referenced this pull request Jan 28, 2021
A part of the microsoft#8415.
Very technical commit to simplify the terminal page code
towards additional steps of simplifying tab management.
No business logic should change.

A firs step in splitting microsoft#8427
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-CodeHealth Issues related to code cleanliness, linting, rules, warnings, errors, static analysis, etc. AutoMerge Marked for automatic merge by the bot when requirements are met Product-Terminal The new Windows Terminal.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants