Skip to content

Commit 7bf0b42

Browse files
committed
fix: Fixed reordering multiple tabs making compact mode hang, b=closes #8978, c=tabs
1 parent 5488b1d commit 7bf0b42

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

src/browser/components/tabbrowser/content/tabs-js.patch

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
diff --git a/browser/components/tabbrowser/content/tabs.js b/browser/components/tabbrowser/content/tabs.js
2-
index 84d633471c89230b981d8a07babef4e0c76c0338..de8b1ecf7cb844f6cf3e66a41b6024c574dfc103 100644
2+
index 84d633471c89230b981d8a07babef4e0c76c0338..a4101ad4e8aa9745ec807eb0373942b53c8dd285 100644
33
--- a/browser/components/tabbrowser/content/tabs.js
44
+++ b/browser/components/tabbrowser/content/tabs.js
55
@@ -83,7 +83,7 @@
@@ -116,6 +116,15 @@ index 84d633471c89230b981d8a07babef4e0c76c0338..de8b1ecf7cb844f6cf3e66a41b6024c5
116116
lastBound
117117
);
118118
} else {
119+
@@ -1168,7 +1192,7 @@
120+
(oldTranslateX && oldTranslateX != newTranslateX) ||
121+
(oldTranslateY && oldTranslateY != newTranslateY);
122+
} else if (this.verticalMode) {
123+
- shouldTranslate &&= oldTranslateY && oldTranslateY != newTranslateY;
124+
+ shouldTranslate &&= oldTranslateY && oldTranslateY != newTranslateY && movingTabs.length === 1;
125+
} else {
126+
shouldTranslate &&= oldTranslateX && oldTranslateX != newTranslateX;
127+
}
119128
@@ -1337,6 +1361,7 @@
120129

121130
let nextItem = this.ariaFocusableItems[newIndex];

src/zen/tabs/ZenPinnedTabManager.mjs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -786,6 +786,7 @@
786786
if (!this.enabled) {
787787
return false;
788788
}
789+
movingTabs = [...movingTabs]
789790
try {
790791
const pinnedTabsTarget =
791792
event.target.closest('.zen-workspace-pinned-tabs-section') ||

0 commit comments

Comments
 (0)