From e572328ab967c6d0d8c1b195996cbbc7988f4853 Mon Sep 17 00:00:00 2001 From: onemen Date: Fri, 1 Nov 2024 11:51:06 +0200 Subject: [PATCH] chore: followup bug 1923052 - Show private browsing indicator icon in navbar when vertical tabs are enabled --- addon/chrome/content/overlay/multirow.css | 8 -------- addon/chrome/content/tab/tab.js | 14 ++++++++++++++ 2 files changed, 14 insertions(+), 8 deletions(-) diff --git a/addon/chrome/content/overlay/multirow.css b/addon/chrome/content/overlay/multirow.css index bcb784f8..7a02c624 100644 --- a/addon/chrome/content/overlay/multirow.css +++ b/addon/chrome/content/overlay/multirow.css @@ -163,14 +163,6 @@ --private-browsing-indicator-margin-top: 9px; } -#TabsToolbar[tabmix-multibar] #private-browsing-indicator-with-label { - align-items: flex-start; - margin-top: var(--private-browsing-indicator-margin-top, 12px); -} - -#TabsToolbar[tabmix-multibar] #private-browsing-indicator-with-label > image { - margin-top: 2px; -} :root:not([uidensity=compact]) #TabsToolbar[tabmix-multibar] .toolbarbutton-1:not([id="tabs-newtab-button"]) { margin-top: var(--tabmix-button-margin-top-proton) !important; diff --git a/addon/chrome/content/tab/tab.js b/addon/chrome/content/tab/tab.js index 343de3ed..6a5527e0 100644 --- a/addon/chrome/content/tab/tab.js +++ b/addon/chrome/content/tab/tab.js @@ -1887,6 +1887,20 @@ window.gTMPprefObserver = { }); } + const selectorType = Tabmix.isVersion(1340) ? "." : "#"; + this.insertRule( + `#TabsToolbar[tabmix-multibar] ${selectorType}private-browsing-indicator-with-label { + align-items: flex-start; + margin-top: var(--private-browsing-indicator-margin-top, 12px); + }` + ); + + this.insertRule( + `#TabsToolbar[tabmix-multibar] ${selectorType}private-browsing-indicator-with-label > image { + margin-top: 2px; + }` + ); + this.dynamicProtonRules(); this.toolbarbuttonTopMargin(); },