Bug 1383473 - Minor Styling fixes in the hamburger menu. r=Gijs draft
authorErica Wright <ewright@mozilla.com>
Wed, 02 Aug 2017 17:03:49 -0400
changeset 621316 f47cad2267a2a865af30fcce42d0f3038bb302f9
parent 619581 52285ea5e54c73d3ed824544cef2ee3f195f05e6
child 640972 dfa605370539179e385b50d5b8d86fe72eeb42da
push id72333
push userbmo:ewright@mozilla.com
push dateFri, 04 Aug 2017 17:14:23 +0000
reviewersGijs
bugs1383473
milestone57.0a1
Bug 1383473 - Minor Styling fixes in the hamburger menu. r=Gijs MozReview-Commit-ID: 2XJau2Yc63m
browser/themes/shared/customizableui/panelUI.inc.css
--- a/browser/themes/shared/customizableui/panelUI.inc.css
+++ b/browser/themes/shared/customizableui/panelUI.inc.css
@@ -357,17 +357,17 @@ photonpanelmultiview panelview {
 }
 
 /* Add 2 * 12px extra width for touch mode button padding. */
 #appMenu-popup[touchmode] panelview {
   min-width: calc(@menuPanelWidth@ + 24px);
 }
 
 photonpanelmultiview .panel-subview-body {
-  margin: 4px 0;
+  padding: 6px 0;
 }
 
 /* END photonpanelview adjustments */
 
 .cui-widget-panel.cui-widget-panelWithFooter > .panel-arrowcontainer > .panel-arrowcontent {
   padding-bottom: 0;
 }
 
@@ -1243,20 +1243,21 @@ panelview:not([mainView]) .subviewbutton
   color: GrayText;
 }
 
 .PanelUI-subView .subviewbutton-nav::after {
   -moz-context-properties: fill;
   content: url(chrome://browser/skin/back-12.svg);
   fill: GrayText;
   float: right;
+  transform: translateY(1px);
 }
 
 .PanelUI-subView .subviewbutton-nav:-moz-locale-dir(ltr)::after {
-  transform: scaleX(-1);
+  transform: scaleX(-1) translateY(1px);
 }
 
 .subviewbutton[shortcut]::after,
 .subviewbutton[shortcut]::after,
 .PanelUI-subView .subviewbutton-nav::after {
   margin-inline-start: 10px;
 }
 
@@ -1429,17 +1430,17 @@ menuitem.subviewbutton@menuStateActive@,
 .widget-overflow-list .toolbarbutton-1@buttonStateActive@,
 .toolbaritem-combined-buttons@inAnyPanel@ > toolbarbutton@buttonStateActive@ {
   background-color: var(--arrowpanel-dimmed-further);
   border-color: var(--panel-separator-color);
   box-shadow: 0 1px 0 hsla(210,4%,10%,.03) inset;
 }
 
 .subviewbutton.panel-subview-footer {
-  margin: 4px -4px -4px;
+  margin: 0;
   background-color: var(--arrowpanel-dimmed);
   border-top: 1px solid var(--panel-separator-color);
   border-radius: 0;
 }
 
 menuitem.panel-subview-footer@menuStateHover@,
 .subviewbutton.panel-subview-footer@buttonStateHover@ {
   background-color: var(--arrowpanel-dimmed-further);
@@ -1522,18 +1523,18 @@ menuitem.panel-subview-footer@menuStateA
   margin-right: 10px;
 }
 
 .PanelUI-subView toolbarseparator[orient="vertical"] {
   height: 24px;
   border-inline-start: 1px solid var(--panel-separator-color);
   border-top: none;
   margin: 0;
-  margin-inline-start: 4px;
-  margin-inline-end: 5px;
+  margin-inline-start: 6px;
+  margin-inline-end: 7px;
 }
 
 .subviewbutton > .menu-accel-container {
   -moz-box-pack: start;
   margin-inline-start: 10px;
   margin-inline-end: auto;
   color: GrayText;
 }