Bug 1401735 - Move the toolbarbutton badge slightly down in the bookmarks toolbar. r=Gijs, a=sledru
authorJohann Hofmann <jhofmann@mozilla.com>
Thu, 21 Sep 2017 11:47:32 +0200
changeset 431824 b9fd8bc899fac076254d80161f025cad198a6595
parent 431823 c270c61e90151e1ad1f94637b6b907c6314db8d1
child 431825 ae054da94c28841293d131354ee937bdf651c6d9
push id7820
push userryanvm@gmail.com
push dateMon, 25 Sep 2017 13:42:55 +0000
treeherdermozilla-beta@8158695b90e4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersGijs, sledru
bugs1401735
milestone57.0
Bug 1401735 - Move the toolbarbutton badge slightly down in the bookmarks toolbar. r=Gijs, a=sledru MozReview-Commit-ID: Hp1t8YWtQ9r
browser/themes/shared/toolbarbuttons.inc.css
--- a/browser/themes/shared/toolbarbuttons.inc.css
+++ b/browser/themes/shared/toolbarbuttons.inc.css
@@ -303,16 +303,26 @@ toolbarbutton.bookmark-item {
 .bookmark-item > .toolbarbutton-text {
   display: -moz-box !important;
 }
 
 #PlacesToolbarItems > .bookmark-item > .toolbarbutton-icon[label]:not([label=""]) {
   margin-inline-end: 4px;
 }
 
+/* The bookmarks toolbar is smaller than the other toolbars, so we
+ * need to override the badge position to not be cut off. */
+#PersonalToolbar .toolbarbutton-badge {
+  margin-top: -1px !important;
+}
+
+:root[uidensity=touch] #PersonalToolbar .toolbarbutton-badge {
+  margin-top: -4px !important;
+}
+
 /* Remove a pixel of margin on the end so that the badge doesn't
  * overflow the toolbar and push the button into the overflow menu. */
 :root[uidensity=compact] .toolbarbutton-badge {
   margin-inline-end: -7px !important;
 }
 
 /* To allow toolbarbuttons in the bookmarks toolbar to grow in
  * height with the toolbar (like bookmark items), we apply background