Bug 1350315 - Hide cursor when video controls fadeout for desktop. r=gijs draft
authorRay Lin <ralin@mozilla.com>
Sat, 25 Mar 2017 11:41:44 +0800
changeset 551564 e13e6e051e7c6cc3c0817b74ae3b3a59a9b7b918
parent 551223 65b0ac174753b22c01156d72fb42d2abd3176dd1
child 621573 7c97b2c983a23b404d400d2b7f81444c59a00efb
push id51084
push userbmo:ralin@mozilla.com
push dateMon, 27 Mar 2017 02:39:38 +0000
reviewersgijs
bugs1350315
milestone55.0a1
Bug 1350315 - Hide cursor when video controls fadeout for desktop. r=gijs We used to have this rule in shared stylesheet for both mobile and desktop, but the namespace mismatched after visual refresh (De-XUL). Now we should add this rule back to hide cursor while fading out. MozReview-Commit-ID: HDZKn8CrQ5X
toolkit/content/widgets/videocontrols.css
toolkit/themes/shared/media/videocontrols.css
--- a/toolkit/content/widgets/videocontrols.css
+++ b/toolkit/content/widgets/videocontrols.css
@@ -25,20 +25,16 @@
   direction: ltr;
   /* Prevent unwanted style inheritance. See bug 554717. */
   text-align: left;
   list-style-image: none !important;
   font: normal normal normal 100%/normal sans-serif !important;
   text-decoration: none !important;
 }
 
-.controlsSpacer[hideCursor] {
-  cursor: none;
-}
-
 .controlsOverlay[scaled] {
   -moz-box-align: center;
 }
 
 /* CSS Transitions
  *
  * These are overriden by the default theme; the rules here just 
  * provide a fallback to drive the required transitionend event
--- a/toolkit/themes/shared/media/videocontrols.css
+++ b/toolkit/themes/shared/media/videocontrols.css
@@ -17,16 +17,20 @@ audio > xul|videocontrols {
 .controlBar[hidden] {
   display: none;
 }
 
 .controlBar[size="hidden"] {
   display: none;
 }
 
+.controlsSpacer[hideCursor] {
+  cursor: none;
+}
+
 .controlsContainer,
 .progressContainer {
   position: relative;
   height: 100%;
 }
 
 .stackItem {
   position: absolute;