Bug 1362100 - Ensure bookmark toolbar button dropmarker icon remains correct size while animating. r=dao draft
authorNihanth Subramanya <nhnt11@gmail.com>
Fri, 05 May 2017 18:43:23 +0530
changeset 573316 310e696cd6a704e7f47ffa3570469eece7b1e0e2
parent 572730 0b255199db9d6a6f189b89b7906f99155bde3726
child 627246 0c956699c014241aa154b6292144ba81e835d704
push id57331
push usernhnt11@gmail.com
push dateFri, 05 May 2017 13:50:11 +0000
reviewersdao
bugs1362100
milestone55.0a1
Bug 1362100 - Ensure bookmark toolbar button dropmarker icon remains correct size while animating. r=dao The actual dropmarker's list-style-image was forced to none while animating, with another one overlayed on top of it. Setting visibility: hidden; accomplishes the goal of hiding the default icon without affecting layout. MozReview-Commit-ID: Q2TOuNBie7
browser/themes/shared/bookmarked-notification.inc.css
--- a/browser/themes/shared/bookmarked-notification.inc.css
+++ b/browser/themes/shared/bookmarked-notification.inc.css
@@ -51,17 +51,17 @@
 
 @media (min-resolution: 2dppx) {
   #bookmarked-notification-anchor[notification="finish"] > #bookmarked-notification {
     background-image: url("chrome://browser/skin/places/bookmarks-notification-finish@2x.png");
   }
 }
 
 #bookmarks-menu-button[notification="finish"] > .toolbarbutton-menubutton-dropmarker > .dropmarker-icon {
-  list-style-image: none !important;
+  visibility: hidden;
 }
 
 #bookmarked-notification-dropmarker-anchor[notification="finish"] > #bookmarked-notification-dropmarker-icon {
   visibility: visible;
   animation: animation-bookmarkPulse 300ms;
   animation-delay: 600ms;
   animation-timing-function: ease-out;
 }