Merge pull request #189 from geeeeeeeeek/issue/close-settings-bug

Fix close settings causing split webviews (#188).
This commit is contained in:
Akash Nimare
2017-06-28 08:48:06 -07:00
committed by GitHub

View File

@@ -70,7 +70,7 @@ class ServerManagerView {
}
openFunctionalTab(tabProps) {
if (this.functionalTabs[tabProps.name]) {
if (this.functionalTabs[tabProps.name] !== undefined) {
this.activateTab(this.functionalTabs[tabProps.name]);
return;
}
@@ -150,7 +150,10 @@ class ServerManagerView {
delete this.tabs[index];
delete this.functionalTabs[name];
this.activateTab(0, false);
// Issue #188: If the functional tab was not focused, do not activate another tab.
if (this.activeTabIndex === index) {
this.activateTab(0, false);
}
}
updateBadge() {