From a1ed98290390202573533eed27ea01144da1d30f Mon Sep 17 00:00:00 2001 From: Matthew Noorenberghe Date: Fri, 21 Mar 2014 15:44:00 -0700 Subject: [PATCH] Bug 878436 - Update Lion Fullscreen window styling offsets to avoid themes shifting position. r=timdream Based on investigation by Nils Maier [:nmaier]. --- browser/themes/osx/browser.css | 22 ++++++++++------------ 1 file changed, 10 insertions(+), 12 deletions(-) diff --git a/browser/themes/osx/browser.css b/browser/themes/osx/browser.css index f9b75370bb01..97476d92ccb1 100644 --- a/browser/themes/osx/browser.css +++ b/browser/themes/osx/browser.css @@ -2765,8 +2765,8 @@ toolbarbutton.chevron > .toolbarbutton-menu-dropmarker { content: ''; display: block; -moz-appearance: toolbar; - height: calc(@tabHeight@ + 1px); - margin-bottom: calc(-1px - @tabHeight@); + height: @tabHeight@; + margin-bottom: -@tabHeight@; visibility: hidden; } @@ -3901,19 +3901,17 @@ toolbarbutton.chevron > .toolbarbutton-menu-dropmarker { /* Lion Fullscreen window styling */ @media (-moz-mac-lion-theme) { #navigator-toolbox[inFullscreen]:not(:-moz-lwtheme)::before { - height: calc(@tabHeight@ + 11px) !important; + /* Adjust by the full element height of #titlebar, since that element is + * not displayed in native full-screen. + * Also add the height of the tabs, since we're calculating the + * total height of this pseudo-element, not just the top-padding. */ + height: calc(@tabHeight@ + @spaceAboveTabbar@) !important; } #main-window[inFullscreen][privatebrowsingmode=temporary], #main-window[inFullscreen]:-moz-lwtheme { - /* This additional padding matches the change in height in the pseudo-element - * above. */ - padding-top: 11px; - } - #main-window[inFullscreen]:not([privatebrowsingmode=temporary]):-moz-lwtheme { - /* In combination with the previous rule, forces the top 22px of the - * background image to be hidden, so the image doesn't jump around with - * the loss of the titlebar. */ - background-position: right -11px; + /* Adjust by the full element height of #titlebar, since that element is + * not displayed in native full-screen. */ + padding-top: @spaceAboveTabbar@; } }