Skip to content

fix tab control and tabs is not synced after focused tab is removed#737

Merged
zadjii-msft merged 2 commits intomicrosoft:masterfrom
bef178:tab-sync
May 14, 2019
Merged

fix tab control and tabs is not synced after focused tab is removed#737
zadjii-msft merged 2 commits intomicrosoft:masterfrom
bef178:tab-sync

Conversation

@bef178
Copy link
Contributor

@bef178 bef178 commented May 13, 2019

fix #708

@zadjii-msft
Copy link
Member

@bef178 Could you make the title more descriptive? "Tab Sync" will be really hard to remember what it means when looking back at the commit log 3 months from now.

@zadjii-msft zadjii-msft added the Issue-Bug It either shouldn't be doing this or needs an investigation. label May 13, 2019
@zadjii-msft zadjii-msft added this to the Windows Terminal v1.0 milestone May 13, 2019
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 approve, conditional on fixing the things @zadjii-msft already commented on. I have the same concerns.

@bef178 bef178 changed the title Tab sync make _tabs and TabView sync after focused tab is removed May 14, 2019
@bef178 bef178 changed the title make _tabs and TabView sync after focused tab is removed sync _tabs and TabView after focused tab is removed May 14, 2019
@bef178 bef178 changed the title sync _tabs and TabView after focused tab is removed fix tab control and tabs is not synced after focused tab is removed May 14, 2019
@zadjii-msft zadjii-msft merged commit ef8e20a into microsoft:master May 14, 2019
@bef178 bef178 deleted the tab-sync branch May 15, 2019 02:36
ObsidianPhoenix pushed a commit to ObsidianPhoenix/Terminal that referenced this pull request May 15, 2019
…icrosoft#737)

* trim duplicated github627

* fix tab control and tabs is not synced after focused tab is removed
@fghzxm fghzxm mentioned this pull request May 21, 2019
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Issue-Bug It either shouldn't be doing this or needs an investigation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect tab behavior after using Ctrl+w

3 participants