Fix #43 where seek bar and volume control feel janky (#45)

Author: mderrick

src/components/controls/mute/Mute.js

@@ -33,7 +33,7 @@ var Mute = React.createClass({
      * @return {undefined}
      */
     changeVolume(e) {
-        this.props.setVolume(e.target.value / 100);
+        this.props.setVolume(e.target.value / 100, true);
         this.props.unmute();
     },
 

src/components/controls/seek/Seek.js

@@ -40,7 +40,7 @@ var Seek = React.createClass({
      * @return {undefined}
      */
     seek(e) {
-        this.props.seek(e.target.value * this.props.duration / 100);
+        this.props.seek(e.target.value * this.props.duration / 100, true);
     },
 
     onFocus() {

src/components/video/Video.js

@@ -196,19 +196,37 @@ var Video = React.createClass({
     /**
      * Seeks the video timeline.
      * @param  {number} time The value in seconds to seek to
+     * @param  {bool}   forceUpdate Forces a state update without waiting for
+     *                              throttled event.          
      * @return {undefined}
      */
-    seek(time) {
+    seek(time, forceUpdate) {
         this.videoEl.currentTime = time;
+        // In some use cases, we wish not to wait for `onSeeked` or `onSeeking`
+        // throttled event to update state so we force it. This is because
+        // this method is often triggered when dragging a bar and can feel janky.
+        // See https://github.com/mderrick/react-html5video/issues/43
+        if (forceUpdate) {
+            this.updateStateFromVideo();
+        }
     },
 
     /**
      * Sets the video volume.
      * @param  {number} volume The volume level between 0 and 1.
+     * @param  {bool}   forceUpdate Forces a state update without waiting for
+     *                              throttled event.  
      * @return {undefined}
      */
-    setVolume(volume) {
+    setVolume(volume, forceUpdate) {
         this.videoEl.volume = volume;
+        // In some use cases, we wish not to wait for `onVolumeChange`
+        // throttled event to update state so we force it. This is because
+        // this method is often triggered when dragging a bar and can feel janky.
+        // See https://github.com/mderrick/react-html5video/issues/43
+        if (forceUpdate) {
+            this.updateStateFromVideo();
+        }
     },
 
     /**