From 97577dac26f5e338e90f8e42b5dd453947d41332 Mon Sep 17 00:00:00 2001 From: "mr. m" <91018726+mr-cheffy@users.noreply.github.com> Date: Thu, 20 Aug 2026 21:18:13 +0200 Subject: [PATCH] gh-15031: Fixed duplicate separator on tab context menu (gh-15035) --- .../tabbrowser/content/tab-context-menu-js.patch | 11 +++++++++-- src/zen/media/ZenMediaController.mjs | 6 +++--- src/zen/tabs/zen-tabs/vertical-tabs.css | 3 +-- 3 files changed, 13 insertions(+), 7 deletions(-) diff --git a/src/browser/components/tabbrowser/content/tab-context-menu-js.patch b/src/browser/components/tabbrowser/content/tab-context-menu-js.patch index 5346d1c4f..a99739423 100644 --- a/src/browser/components/tabbrowser/content/tab-context-menu-js.patch +++ b/src/browser/components/tabbrowser/content/tab-context-menu-js.patch @@ -1,8 +1,8 @@ diff --git a/browser/components/tabbrowser/content/tab-context-menu.js b/browser/components/tabbrowser/content/tab-context-menu.js -index 4a7cb33a4d9067d469bbe09d4cd8844581642f7a..2e2404b78deb428e57d01b8b929231f675a21323 100644 +index 4a7cb33a4d9067d469bbe09d4cd8844581642f7a..d7f75471a207cccac6ca3dec49037ac30dd76163 100644 --- a/browser/components/tabbrowser/content/tab-context-menu.js +++ b/browser/components/tabbrowser/content/tab-context-menu.js -@@ -839,7 +839,8 @@ var TabContextMenu = { +@@ -839,13 +839,15 @@ var TabContextMenu = { ); contextUnpinSelectedTabs.hidden = !this.contextTab.pinned || !this.multiselected; @@ -12,3 +12,10 @@ index 4a7cb33a4d9067d469bbe09d4cd8844581642f7a..2e2404b78deb428e57d01b8b929231f6 // Build Ask Chat items. The classic layout shows the full ask-chat submenu // (#context_askChat); the alt layout shows only a flat "Summarize Page" item // (#context_askChatSummarize). Hide whichever one this layout doesn't use so + // a pref toggle reverses cleanly. + if (!this._altTabContextMenu) { + document.getElementById("context_askChatSummarize").hidden = true; ++ document.getElementById("context_aiSeparator").hidden = true; + TabContextMenu.GenAI.buildTabMenu( + document.getElementById("context_askChat"), + this diff --git a/src/zen/media/ZenMediaController.mjs b/src/zen/media/ZenMediaController.mjs index eacee1cf1..fe0e927f1 100644 --- a/src/zen/media/ZenMediaController.mjs +++ b/src/zen/media/ZenMediaController.mjs @@ -346,7 +346,7 @@ class ZenMediaCard { onToggle() { if (this.controller?.isPlaying) { - this.controller.pause(); + this.controller.pause("user"); } else { this.controller?.play(); } @@ -365,7 +365,7 @@ class ZenMediaCard { } onSeekDrag(event) { - this.controller?.pause(); + this.controller?.pause("user"); const newTime = (event.target.value / 100) * this.duration; this.currentTimeEl.textContent = this.formatSecondsToTime(newTime); } @@ -408,7 +408,7 @@ class ZenMediaCard { onClose() { if (this.controller) { - this.controller.pause(); + this.controller.pause("user"); } else { const webRTC = this.browser.browsingContext.currentWindowGlobal.getActor("WebRTC"); diff --git a/src/zen/tabs/zen-tabs/vertical-tabs.css b/src/zen/tabs/zen-tabs/vertical-tabs.css index 8cc0975f6..c27580403 100644 --- a/src/zen/tabs/zen-tabs/vertical-tabs.css +++ b/src/zen/tabs/zen-tabs/vertical-tabs.css @@ -538,8 +538,7 @@ :root[zen-single-toolbar="true"]:not([zen-has-empty-tab="true"]) & { & #urlbar:not([breakout-extend]) .urlbar-input-container { - padding-left: 8px; - padding-right: 4px; + padding-inline: 4px; } } }