Bug 1439834 - Stretch #titlebar over dragspace. r=dao, a=jcristau
authorMartin Stransky <stransky@redhat.com>
Wed, 07 Mar 2018 10:58:17 +0100
changeset 460357 115138a61e50a81f19982f0d5c3e33a555bdbcb0
parent 460356 48151778db661b967db3f8e6ebca118752e0b203
child 460358 3d541b457493ffa308b544dc48dfaa29b37991b5
push id8921
push userryanvm@gmail.com
push dateMon, 26 Mar 2018 14:40:36 +0000
treeherdermozilla-beta@a15c367a272e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdao, jcristau
bugs1439834
milestone60.0
Bug 1439834 - Stretch #titlebar over dragspace. r=dao, a=jcristau Add missing drag space height (present as #TabsToolbar padding) to titlebar height computation. MozReview-Commit-ID: 8JhP2zuNexb
browser/themes/linux/browser.css
--- a/browser/themes/linux/browser.css
+++ b/browser/themes/linux/browser.css
@@ -655,25 +655,25 @@ notification[value="translation"] menuli
   }
   :root[tabsintitlebar][sizemode="maximized"] > #titlebar {
     -moz-appearance: -moz-window-titlebar-maximized;
   }
 
   /* Add extra space to titlebar for dragging */
   :root[sizemode="normal"][chromehidden~="menubar"] #TabsToolbar,
   :root[sizemode="normal"] #toolbar-menubar[autohide="true"][inactive] + #TabsToolbar {
-    margin-top: var(--space-above-tabbar);
+    padding-top: var(--space-above-tabbar);
   }
 
   /* Private browsing and accessibility indicators */
   :root[sizemode="normal"][chromehidden~="menubar"] #TabsToolbar > .private-browsing-indicator,
   :root[sizemode="normal"][chromehidden~="menubar"] #TabsToolbar > .accessibility-indicator,
   :root[sizemode="normal"] #toolbar-menubar[autohide="true"][inactive] + #TabsToolbar > .private-browsing-indicator,
   :root[sizemode="normal"] #toolbar-menubar[autohide="true"][inactive] + #TabsToolbar > .accessibility-indicator {
-    margin-top: calc(-1 * var(--space-above-tabbar));
+    padding-top: calc(-1 * var(--space-above-tabbar));
   }
 
   /* The button box must appear on top of the navigator-toolbox in order for
    * click and hover mouse events to work properly for the button in the restored
    * window state. Otherwise, elements in the navigator-toolbox, like the menubar,
    * can swallow those events.
    */
   #titlebar-buttonbox {