allow poster image to toggle playback

This allows the poster image to toggle play / pause on audio-only sources
videojs · Apr 24, 2015 · adc5415 · adc5415
1 parent 697325e
commit adc5415
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 4 deletions.
diff --git a/sandbox/icons.html.example b/sandbox/icons.html.example
@@ -4,7 +4,7 @@
   <meta charset="utf-8" />
   <title>Video.js Icons Sandbox</title>
 
-  <link href="../build/files/video-js.css" rel="stylesheet" type="text/css">
+  <link href="../build/temp/video-js.css" rel="stylesheet" type="text/css">
 
   <style>
   .icon-list li span { font-size: 150% }

diff --git a/sandbox/index.html.example b/sandbox/index.html.example
@@ -6,12 +6,11 @@
 
   <link href="../build/temp/video-js.css" rel="stylesheet" type="text/css">
 
-  <!-- LOAD VIDEO.JS SOURCE FILES IN ORDER -->
   <script src="../build/temp/video.js"></script>
 
   <!-- Set the location of the flash SWF -->
   <script>
-    videojs.options.flash.swf = '../build/temp/video-js.swf'
+    videojs.options.flash.swf = '../build/temp/video-js.swf';
   </script>
 
 </head>

diff --git a/src/css/components/_control-bar.scss b/src/css/components/_control-bar.scss
@@ -36,6 +36,12 @@
   display: none;
 }
 
+// Don't hide the control bar if it's audio
+.video-js.vjs-audio.vjs-has-started.vjs-user-inactive.vjs-playing .vjs-control-bar {
+  opacity: 1;
+  visibility: visible;
+}
+
 /* IE8 is flakey with fonts, and you have to change the actual content to force
 fonts to show/hide properly.
 - "\9" IE8 hack didn't work for this

diff --git a/src/js/poster-image.js b/src/js/poster-image.js
@@ -92,7 +92,11 @@ class PosterImage extends Button {
   onClick() {
     // We don't want a click to trigger playback when controls are disabled
     // but CSS should be hiding the poster to prevent that from happening
-    this.player_.play();
+    if (this.player_.paused()) {
+      this.player_.play();
+    } else {
+      this.player_.pause();
+    }
   }
 
 }