tor-browser

commit 25277aecc63305c6a77983c658363a15a4676563
parent c6c09de1ebae1780fb607498436b8d12221718e7
Author: Kyler Riggs <100742516+ky-ler@users.noreply.github.com>
Date:   Wed,  8 Oct 2025 14:54:00 +0000

Bug 1974265 - Add "Turn on Vertical Tabs" option to titlebar spacers context menu. r=nsharpley,dao

Differential Revision: https://phabricator.services.mozilla.com/D267155

Diffstat:
M
browser/components/customizableui/ToolbarContextMenu.sys.mjs
 | 
7
++++++-
M
browser/components/customizableui/test/browser_customization_context_menus.js
 | 
3
+++

2 files changed, 9 insertions(+), 1 deletion(-)
diff --git a/browser/components/customizableui/ToolbarContextMenu.sys.mjs b/browser/components/customizableui/ToolbarContextMenu.sys.mjs
@@ -232,6 +232,9 @@ export var ToolbarContextMenu = {
     let removeFromToolbar = popup.querySelector(
       ".customize-context-removeFromToolbar"
     );
+
+    let isTitlebarSpacer = toolbarItem?.classList.contains("titlebar-spacer");
+
     // Show/hide fullscreen context menu items and set the
     // autohide item's checked state to mirror the autohide pref.
     showFullScreenViewContextMenuItems(popup);
@@ -240,7 +243,9 @@ export var ToolbarContextMenu = {
     let sidebarRevampEnabled = Services.prefs.getBoolPref("sidebar.revamp");
     let showSidebarActions =
       ["tabbrowser-tabs", "sidebar-button"].includes(toolbarItem?.id) ||
-      toolbarItem?.localName == "toolbarspring";
+      toolbarItem?.localName == "toolbarspring" ||
+      isTitlebarSpacer;
+
     let toggleVerticalTabsItem = document.getElementById(
       "toolbar-context-toggle-vertical-tabs"
     );
diff --git a/browser/components/customizableui/test/browser_customization_context_menus.js b/browser/components/customizableui/test/browser_customization_context_menus.js
@@ -139,6 +139,7 @@ add_task(async function titlebar_spacer_context() {
   let spacer = document.querySelector(
     "#TabsToolbar .titlebar-spacer[type='pre-tabs']"
   );
+  await waitForElementShown(spacer);
   EventUtils.synthesizeMouseAtCenter(spacer, {
     type: "contextmenu",
     button: 2,
@@ -146,6 +147,8 @@ add_task(async function titlebar_spacer_context() {
   await shownPromise;
 
   let expectedEntries = [
+    ["#toolbar-context-toggle-vertical-tabs", true],
+    ["---"],
     [".customize-context-moveToPanel", false],
     [".customize-context-removeFromToolbar", false],
     ["---"],