Bug 1435993 - move tab-min-height to relevant elements instead of having it live on :root, r?johannh draft
authorGijs Kruitbosch <gijskruitbosch@gmail.com>
Tue, 06 Feb 2018 15:33:07 +0000
changeset 751516 5f4bcf436f4f3c69c5045dc64bf60e31a0886213
parent 751476 f1a4b64f19b0e93c49492735db30a5023e624ae7
push id97997
push userbmo:gijskruitbosch+bugs@gmail.com
push dateTue, 06 Feb 2018 15:33:56 +0000
reviewersjohannh
bugs1435993
milestone60.0a1
Bug 1435993 - move tab-min-height to relevant elements instead of having it live on :root, r?johannh MozReview-Commit-ID: J3Yq3WGzx2A
browser/themes/shared/tabs.inc.css
--- a/browser/themes/shared/tabs.inc.css
+++ b/browser/themes/shared/tabs.inc.css
@@ -1,35 +1,37 @@
 %if 0
 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 %endif
 %filter substitution
 %define horizontalTabPadding 9px
 
-/* This can't live on the tabbrowser because it's used for the titlebar height, too. */
-:root {
+#titlebar,
+#tabbrowser-tabs {
   --tab-min-height: 33px;
 }
 
+:root[uidensity=compact] #titlebar,
+:root[uidensity=compact] #tabbrowser-tabs {
+  --tab-min-height: 29px;
+}
+
+:root[uidensity=touch] #titlebar,
+:root[uidensity=touch] #tabbrowser-tabs {
+  --tab-min-height: 41px;
+}
+
 #tabbrowser-tabs {
   --tabs-top-border-width: 0px;
   --tab-min-width: 76px;
   --tab-loading-fill: #0A84FF;
 }
 
-:root[uidensity=compact] {
-  --tab-min-height: 29px;
-}
-
-:root[uidensity=touch] {
-  --tab-min-height: 41px;
-}
-
 #tabbrowser-tabs:-moz-lwtheme {
   --tab-line-color: var(--lwt-accent-color);
 }
 
 tabbrowser {
   /* Value for --in-content-page-background in in-content/common.inc.css */
   background-color: #f9f9fa;
 }