chore: followup bug 1923052 - Show private browsing indicator icon in…

… navbar when vertical tabs are enabled
onemen · Nov 2, 2024 · e572328 · e572328
1 parent 59d70d0
commit e572328
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 8 deletions.
diff --git 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
@@ -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();
   },