Bug 1350315 - Hide cursor when video controls fadeout for desktop. r=Gijs a=gchang
authorRay Lin <ralin@mozilla.com>
Sat, 25 Mar 2017 11:41:44 +0800
changeset 379297 f758f44a68deac39fe3bc178f1f4da0ea0118b48
parent 379296 6c66fac00e392e08121d905e27e461bdccd853ea
child 379298 f694d1ed31afaed6e87859ae78931f896c48b1dd
push id1419
push userjlund@mozilla.com
push dateMon, 10 Apr 2017 20:44:07 +0000
treeherdermozilla-release@5e6801b73ef6 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersGijs, gchang
bugs1350315
milestone53.0
Bug 1350315 - Hide cursor when video controls fadeout for desktop. r=Gijs a=gchang 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;