From: Robert Kaiser <kairo@kairo.at>
Date: Sun, 31 Aug 2014 19:37:34 +0000 (+0200)
Subject: apply actually working rules to cover most of mac (except non-graphite 10.6) until... 
X-Git-Tag: LCARStrek-2.29~10
X-Git-Url: https://git-public.kairo.at/?p=themes.git;a=commitdiff_plain;h=ee49f520102b39e3559c8d4fda2f3487e5195038

apply actually working rules to cover most of mac (except non-graphite 10.6) until bug 1060941 hopefully will get a proper fix in place
---

diff --git a/LCARStrek/browser/browser.css b/LCARStrek/browser/browser.css
index 48225130..ab4d54d2 100644
--- a/LCARStrek/browser/browser.css
+++ b/LCARStrek/browser/browser.css
@@ -17,8 +17,8 @@ toolbar[customizable="true"] {
   -moz-binding: url("chrome://global/skin/globalBindings.xml#toolbar-browsercustom");
 }
 
-@supports ( -moz-appearance: -moz-mac-unified-toolbar ) {
-  /* This is a usable workaround to detect we're on Mac OS X. */
+@media (-moz-mac-graphite-theme), (-moz-mac-lion-theme), (-moz-mac-yosemite-theme) {
+  /* This is a hackish workaround to cover most of Mac OS until bug 1060941 gets fixed. */
   toolbar[customizable="true"]:not([nowindowdrag="true"]) {
     -moz-binding: url("chrome://global/skin/globalBindings.xml#toolbar-drag-browsercustom");
   }
@@ -74,15 +74,15 @@ menubar {
 /*  margin-top: 15px;*/
 }
 
-@supports ( not(-moz-appearance: -moz-mac-unified-toolbar) ) {
-  /* This is a usable workaround to detect we're not on Mac OS X. */
+@media (-moz-mac-graphite-theme: 0) and (-moz-mac-lion-theme: 0) {
+  /* This is a hackish workaround to exclude most of Mac OS until bug 1060941 gets fixed. */
   #toolbar-menubar:not([autohide="true"]) {
     -moz-binding: url("chrome://global/skin/globalBindings.xml#toolbar-drag-browsercustom");
   }
 }
 
-@supports ( -moz-appearance: -moz-mac-unified-toolbar ) {
-  /* This is a usable workaround to detect we're on Mac OS X. */
+@media (-moz-mac-graphite-theme), (-moz-mac-lion-theme) {
+  /* This is a hackish workaround to cover most of Mac OS until bug 1060941 gets fixed. */
   #main-window[tabsintitlebar]:not([inFullscreen="true"]) #TabsToolbar {
     padding-left: 60px;
   }
@@ -113,9 +113,9 @@ menubar {
 #navigator-toolbox > toolbar:not(:-moz-lwtheme) {
 }
 
-@supports ( not(-moz-appearance: -moz-mac-unified-toolbar) ) {
-  /* This is a usable workaround to detect we're not on Mac OS X. */
-  /* indent also due to non-applicable aero rule */
+@media (-moz-mac-graphite-theme: 0) and (-moz-mac-lion-theme: 0) {
+  /* This is a hackish workaround to exclude most of Mac OS until bug 1060941 gets fixed. */
+  /* Indent also due to non-applicable aero rule in original Windows theme. */
   #toolbar-menubar {
     background-color: transparent !important;
   }
@@ -151,8 +151,8 @@ menubar {
   }
 }
 
-@supports ( -moz-appearance: -moz-mac-unified-toolbar ) {
-  /* This is a usable workaround to detect we're on Mac OS X. */
+@media (-moz-mac-graphite-theme), (-moz-mac-lion-theme) {
+  /* This is a hackish workaround to cover most of Mac OS until bug 1060941 gets fixed. */
   #titlebar-buttonbox > .titlebar-button {
     display: none;
   }
@@ -196,8 +196,8 @@ menubar {
   margin-top: 4px;
 }
 
-@supports ( -moz-appearance: -moz-mac-unified-toolbar ) {
-  /* This is a usable workaround to detect we're on Mac OS X. */
+@media (-moz-mac-graphite-theme), (-moz-mac-lion-theme) {
+  /* This is a hackish workaround to cover most of Mac OS until bug 1060941 gets fixed. */
   #main-window[sizemode="normal"] > #titlebar,
   #main-window[sizemode="maximized"] > #titlebar {
     margin-top: 19px;
@@ -261,8 +261,8 @@ menubar {
   list-style-image: url("chrome://navigator/skin/icons/win-close-hover.gif");
 }
 
-@supports ( -moz-appearance: -moz-mac-unified-toolbar ) {
-  /* This is a usable workaround to detect we're on Mac OS X. */
+@media (-moz-mac-graphite-theme), (-moz-mac-lion-theme) {
+  /* This is a hackish workaround to cover most of Mac OS until bug 1060941 gets fixed. */
   #titlebar-fullscreen-button {
     -moz-appearance: none;
     list-style-image: url("chrome://browser/skin/icons/mac-fullscreen-icon.png");
diff --git a/LCARStrek/global/global.css b/LCARStrek/global/global.css
index 537e261b..6cf895d0 100644
--- a/LCARStrek/global/global.css
+++ b/LCARStrek/global/global.css
@@ -48,7 +48,7 @@ toolbaritem > menubar {
   -moz-binding: url("chrome://global/content/bindings/toolbar.xml#menubar");
 }
 
-@media (-moz-mac-graphite-theme), (-moz-mac-lion-theme), (-moz-mac-yosemite-theme) {
+@media (-moz-mac-graphite-theme), (-moz-mac-lion-theme) {
   /* This is a hackish workaround to cover most of Mac OS until bug 1060941 gets fixed. */
   toolbar.chromeclass-menubar {
     -moz-binding: url("chrome://global/content/bindings/toolbar.xml#toolbar");
@@ -56,15 +56,6 @@ toolbaritem > menubar {
   }
 }
 
-/* HACK TEST ONLY: try several mac supports rules! */
-
-@media (-moz-mac-graphite-theme: 0) and (-moz-mac-lion-theme: 0) and (-moz-mac-yosemite-theme: 0) {
-  /* this is a usable workaround to detect we're on Mac OS */
-  toolbox {
-    border: 1px solid blue !important;
-  }
-}
-
 /* ::::: root elements ::::: */
 
 window,