Bug 1376893 - Don't display the disabled reload button with less opacity since it is only disabled temporarily and causes flickering in the UI. r?johannh draft
authorJared Wein <jwein@mozilla.com>
Wed, 28 Jun 2017 13:16:21 -0700
changeset 602426 50b95337f82c2e1b39aaca6591a2a7253bc6c980
parent 600694 8f80d594c08d5c7a112e5d4b9eb44ffca717eb7b
child 635603 854211fe352122498507dd26eb557fc8373b2563
push id66435
push userbmo:jaws@mozilla.com
push dateFri, 30 Jun 2017 00:30:32 +0000
reviewersjohannh
bugs1376893
milestone56.0a1
Bug 1376893 - Don't display the disabled reload button with less opacity since it is only disabled temporarily and causes flickering in the UI. r?johannh MozReview-Commit-ID: FHbzB54SSqC
browser/base/content/browser.css
--- a/browser/base/content/browser.css
+++ b/browser/base/content/browser.css
@@ -423,16 +423,24 @@ toolbar:not(#TabsToolbar) > #personal-bo
   display: -moz-box;
 }
 
 #reload-button:not([displaystop]) + #stop-button,
 #reload-button[displaystop] {
   visibility: collapse;
 }
 
+/* The reload-button is only disabled temporarily when it becomes visible
+   to prevent users from accidentally clicking it. We don't however need
+   to show this disabled state, as the flicker that it generates is short
+   enough to be visible but not long enough to explain anything to users. */
+#reload-button[disabled]:not(:-moz-window-inactive) > .toolbarbutton-icon {
+  opacity: 1 !important;
+}
+
 #PanelUI-feeds > .feed-toolbarbutton:-moz-locale-dir(rtl) {
   direction: rtl;
 }
 
 #panelMenu_bookmarksMenu > .bookmark-item {
   max-width: none;
 }