Bug 1383535 - fix margin issue with sync item when not signed in, r=mossop
authorGijs Kruitbosch <gijskruitbosch@gmail.com>
Wed, 26 Jul 2017 17:39:48 +0100
changeset 419821 93d48f1320f774e6bd2f96d6405f25b67f8f8850
parent 419820 e18f2b95fc8ed65a226488bdd0f9d46ed9e40caa
child 419822 b6da3839bc2acc8cfc038a5b6813aad899a4c67c
push id7566
push usermtabara@mozilla.com
push dateWed, 02 Aug 2017 08:25:16 +0000
treeherdermozilla-beta@86913f512c3c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmossop
bugs1383535
milestone56.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1383535 - fix margin issue with sync item when not signed in, r=mossop MozReview-Commit-ID: HJk2rBBpY3P
browser/themes/shared/customizableui/panelUI.inc.css
--- a/browser/themes/shared/customizableui/panelUI.inc.css
+++ b/browser/themes/shared/customizableui/panelUI.inc.css
@@ -746,17 +746,20 @@ toolbaritem[cui-areatype="menu-panel"][s
   padding-inline-start: 10px;
 }
 
 .panel-banner-item > .toolbarbutton-text {
   width: 0; /* Fancy cropping solution for flexbox. */
 }
 
 /* FxAccount indicator bits. */
-#appMenu-fxa-container:not([fxastatus="signedin"]) {
+
+/* Add the .toolbaritem-combined-buttons class to increase the specificity so as
+ * to override the end margin for .toolbaritem-combined-buttons items further down. */
+#appMenu-fxa-container.toolbaritem-combined-buttons:not([fxastatus="signedin"]) {
   margin-inline-end: 0;
 }
 
 #appMenu-fxa-label,
 #appMenu-fxa-icon {
   -moz-image-region: rect(0, 16px, 16px, 0);
   list-style-image: url(chrome://browser/skin/sync-horizontalbar.png);
 }