Skip to content

Commit 5ac63e3

Browse files
committed
feat: Prevent flashing when cancelling new split view creation, b=no-bug, c=split-view
1 parent d7d5d11 commit 5ac63e3

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

src/zen/split-view/ZenViewSplitter.mjs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -212,6 +212,7 @@ class nsZenViewSplitter extends nsZenDOMOperatedFeature {
212212
this.removeGroup(groupIndex);
213213
if (changeTab) {
214214
gBrowser.selectedTab = remainingTabs[remainingTabs.length - 1];
215+
document.getElementById('cmd_zenNewEmptySplit').removeAttribute('disabled');
215216
}
216217
} else {
217218
const node = this.getSplitNodeFromTab(tab);
@@ -1950,7 +1951,7 @@ class nsZenViewSplitter extends nsZenDOMOperatedFeature {
19501951
const groupIndex = this._data.findIndex((group) => group.tabs.includes(emptyTab));
19511952
const newSelectedTab = gBrowser.selectedTab;
19521953
const cleanup = () => {
1953-
this.removeTabFromGroup(emptyTab, groupIndex, { changeTab: false });
1954+
this.removeTabFromGroup(emptyTab, groupIndex, { changeTab: !onSwitch });
19541955
const command = document.getElementById('cmd_zenNewEmptySplit');
19551956
command.removeAttribute('disabled');
19561957
};
@@ -1969,9 +1970,6 @@ class nsZenViewSplitter extends nsZenDOMOperatedFeature {
19691970
this.resetTabState(emptyTab, false);
19701971
this.splitTabs([selectedTab, newSelectedTab], 'grid', 1);
19711972
} else {
1972-
if (!onSwitch) {
1973-
gBrowser.selectedTab = selectedTab;
1974-
}
19751973
cleanup();
19761974
}
19771975
},

0 commit comments

Comments
 (0)