From ff05419df1e084b86d151e7988c374cfdf7887aa Mon Sep 17 00:00:00 2001
From: streamich <streamich@gmail.com>
Date: Mon, 29 Oct 2018 21:12:26 +0100
Subject: [PATCH 1/2] =?UTF-8?q?feat:=20=F0=9F=8E=B8=20add=20useVideo=20hoo?=
 =?UTF-8?q?k?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 README.md                          |   1 +
 docs/useAudio.md                   |   2 +-
 docs/useVideo.md                   |  87 ++++++++++++++
 src/__stories__/useVideo.story.tsx |  36 ++++++
 src/index.ts                       |   2 +
 src/useAudio.ts                    | 176 +---------------------------
 src/useVideo.ts                    |   5 +
 src/util/createHTMLMediaHook.ts    | 182 +++++++++++++++++++++++++++++
 8 files changed, 316 insertions(+), 175 deletions(-)
 create mode 100644 docs/useVideo.md
 create mode 100644 src/__stories__/useVideo.story.tsx
 create mode 100644 src/useVideo.ts
 create mode 100644 src/util/createHTMLMediaHook.ts

diff --git a/README.md b/README.md
index 2270830f..637eab3c 100644
--- a/README.md
+++ b/README.md
@@ -49,6 +49,7 @@
 - [**UI**](./docs/UI.md)
   - [`useAudio`](./docs/useAudio.md) &mdash; plays audio and exposes its controls. [![][img-demo]](https://codesandbox.io/s/5v7q47knwl)
   - [`useSpeech`](./docs/useSpeech.md) &mdash; synthesizes speech from a text string. [![][img-demo]](https://codesandbox.io/s/n090mqz69m)
+  - [`useVideo`](./docs/useVideo.md) &mdash; plays video, tracks its state, and exposes playback controls.
     <br/>
     <br/>
 - [**Animations**](./docs/Animations.md)
diff --git a/docs/useAudio.md b/docs/useAudio.md
index 206681a7..6f74730c 100644
--- a/docs/useAudio.md
+++ b/docs/useAudio.md
@@ -1,6 +1,6 @@
 # `useAudio`
 
-Creates `<audio>` element, tracks its state and exposes playback conrols.
+Creates `<audio>` element, tracks its state and exposes playback controls.
 
 
 ## Usage
diff --git a/docs/useVideo.md b/docs/useVideo.md
new file mode 100644
index 00000000..14f0e5c0
--- /dev/null
+++ b/docs/useVideo.md
@@ -0,0 +1,87 @@
+# `useVideo`
+
+Creates `<video>` element, tracks its state and exposes playback controls.
+
+
+## Usage
+
+```jsx
+import {useVideo} from 'react-use';
+
+const Demo = () => {
+  const [video, state, controls, ref] = useVideo({
+    src: 'http://clips.vorwaerts-gmbh.de/big_buck_bunny.mp4',
+    autoPlay: true,
+  });
+
+  return (
+    <div>
+      {video}
+      <pre>{JSON.stringify(state, null, 2)}</pre>
+      <button onClick={controls.pause}>Pause</button>
+      <button onClick={controls.play}>Play</button>
+      <br/>
+      <button onClick={controls.mute}>Mute</button>
+      <button onClick={controls.unmute}>Un-mute</button>
+      <br/>
+      <button onClick={() => controls.volume(.1)}>Volume: 10%</button>
+      <button onClick={() => controls.volume(.5)}>Volume: 50%</button>
+      <button onClick={() => controls.volume(1)}>Volume: 100%</button>
+      <br/>
+      <button onClick={() => controls.seek(state.time - 5)}>-5 sec</button>
+      <button onClick={() => controls.seek(state.time + 5)}>+5 sec</button>
+    </div>
+  );
+};
+```
+
+
+## Reference
+
+```ts
+const [video, state, controls, ref] = useAudio(props);
+```
+
+`video` is React's `<video>` element that you have to insert somewhere in your
+render tree, for example:
+
+```jsx
+<div>{video}</div>
+```
+
+`state` tracks the state of the video and has the following shape:
+
+```json
+{
+  "buffered": [
+    {
+      "start": 0,
+      "end": 425.952625
+    }
+  ],
+  "time": 5.244996,
+  "duration": 425.952625,
+  "isPlaying": false,
+  "muted": false,
+  "volume": 1
+}
+```
+
+`controls` is a list collection of methods that allow you to control the
+playback of the video, it has the following interface:
+
+```ts
+interface AudioControls {
+  play: () => Promise<void> | void;
+  pause: () => void;
+  mute: () => void;
+  unmute: () => void;
+  volume: (volume: number) => void;
+  seek: (time: number) => void;
+}
+```
+
+`ref` is a React reference to HTML `<video>` element, you can access the element by
+`ref.current`, note that it may be `null`.
+
+And finally, `props` &mdash; all props that `<video>` accepts.
diff --git a/src/__stories__/useVideo.story.tsx b/src/__stories__/useVideo.story.tsx
new file mode 100644
index 00000000..1b3e3e7a
--- /dev/null
+++ b/src/__stories__/useVideo.story.tsx
@@ -0,0 +1,36 @@
+import * as React from 'react';
+import {storiesOf} from '@storybook/react';
+import {useVideo} from '..';
+import ShowDocs from '../util/ShowDocs';
+
+const Demo = () => {
+  const [video, state, controls, ref] = useVideo({
+    src: 'http://clips.vorwaerts-gmbh.de/big_buck_bunny.mp4',
+    autoPlay: true,
+  });
+
+  return (
+    <div>
+      {video}
+      <pre>{JSON.stringify(state, null, 2)}</pre>
+      <button onClick={controls.pause}>Pause</button>
+      <button onClick={controls.play}>Play</button>
+      <br/>
+      <button onClick={controls.mute}>Mute</button>
+      <button onClick={controls.unmute}>Un-mute</button>
+      <br/>
+      <button onClick={() => controls.volume(.1)}>Volume: 10%</button>
+      <button onClick={() => controls.volume(.5)}>Volume: 50%</button>
+      <button onClick={() => controls.volume(1)}>Volume: 100%</button>
+      <br/>
+      <button onClick={() => controls.seek(state.time - 5)}>-5 sec</button>
+      <button onClick={() => controls.seek(state.time + 5)}>+5 sec</button>
+    </div>
+  );
+};
+
+storiesOf('useVideo', module)
+  .add('Docs', () => <ShowDocs md={require('../../docs/useVideo.md')} />)
+  .add('Demo', () =>
+    <Demo/>
+  )
diff --git a/src/index.ts b/src/index.ts
index 679c0d34..171e825f 100644
--- a/src/index.ts
+++ b/src/index.ts
@@ -36,6 +36,7 @@ import useToggle from './useToggle';
 import useTween from './useTween';
 import useUnmount from './useUnmount';
 import useUpdate from './useUpdate';
+import useVideo from './useVideo';
 import useWindowSize from './useWindowSize';
 
 export {
@@ -77,5 +78,6 @@ export {
   useTween,
   useUnmount,
   useUpdate,
+  useVideo,
   useWindowSize,
 };
diff --git a/src/useAudio.ts b/src/useAudio.ts
index f0df39a3..b5c0b3e4 100644
--- a/src/useAudio.ts
+++ b/src/useAudio.ts
@@ -1,177 +1,5 @@
-import * as React from 'react';
-import {useEffect, useRef, ReactRef} from './react';
-import useSetState from './useSetState';
-import parseTimeRanges from './util/parseTimeRanges';
+import createHTMLMediaHook from './util/createHTMLMediaHook';
 
-export interface AudioProps extends React.AudioHTMLAttributes<any> {
-  src: string;
-}
-
-export interface AudioState {
-  buffered: any[];
-  duration: number;
-  isPlaying: boolean;
-  muted: boolean;
-  time: number;
-  volume: number;
-}
-
-export interface AudioControls {
-  play: () => Promise<void> | void;
-  pause: () => void;
-  mute: () => void;
-  unmute: () => void;
-  volume: (volume: number) => void;
-  seek: (time: number) => void;
-}
-
-const useAudio = (props: AudioProps): [React.ReactElement<AudioProps>, AudioState, AudioControls, ReactRef<HTMLAudioElement | null>] => {
-  const [state, setState] = useSetState<AudioState>({
-    buffered: [],
-    time: 0,
-    duration: 0,
-    isPlaying: false,
-    muted: false,
-    volume: 1,
-  });
-  const ref = useRef<HTMLAudioElement | null>(null);
-
-  const wrapEvent = (userEvent, proxyEvent?) => {
-    return (event) => {
-      try {
-        proxyEvent && proxyEvent(event);
-      } finally {
-        userEvent && userEvent(event);
-      }
-    };
-  };
-
-  const onPlay = () => setState({isPlaying: true});
-  const onPause = () => setState({isPlaying: false});
-  const onVolumeChange = () => {
-    const el = ref.current;
-    if (!el) return;
-    setState({
-      muted: el.muted,
-      volume: el.volume,
-    });
-  };
-  const onDurationChange = () => {
-    const el = ref.current;
-    if (!el) return;
-    const {duration, buffered} = el;
-    setState({
-      duration,
-      buffered: parseTimeRanges(buffered),
-    });
-  };
-  const onTimeUpdate = () => {
-    const el = ref.current;
-    if (!el) return;
-    setState({time: el.currentTime});
-  };
-  const onProgress = () => {
-    const el = ref.current;
-    if (!el) return;
-    setState({buffered: parseTimeRanges(el.buffered)});
-  };
-
-  const element = React.createElement('audio', {
-    controls: false,
-    ...props,
-    ref,
-    onPlay: wrapEvent(props.onPlay, onPlay),
-    onPause: wrapEvent(props.onPause, onPause),
-    onVolumeChange: wrapEvent(props.onVolumeChange, onVolumeChange),
-    onDurationChange: wrapEvent(props.onDurationChange, onDurationChange),
-    onTimeUpdate: wrapEvent(props.onTimeUpdate, onTimeUpdate),
-    onProgress: wrapEvent(props.onProgress, onProgress),
-  });
-
-  // Some browsers return `Promise` on `.play()` and may throw errors
-  // if one tries to execute another `.play()` or `.pause()` while that
-  // promise is resolving. So we prevent that with this lock.
-  // See: https://bugs.chromium.org/p/chromium/issues/detail?id=593273
-  let lockPlay: boolean = false;
-
-  const controls = {
-    play: () => {
-      const el = ref.current;
-      if (!el) return undefined;
-
-      if (!lockPlay) {
-        const promise = el.play();
-        const isPromise = typeof promise === 'object';
-
-        if (isPromise) {
-          lockPlay = true;
-          const resetLock = () => {
-            lockPlay = false;
-          };
-          promise.then(resetLock, resetLock);
-        }
-
-        return promise;
-      }
-      return undefined;
-    },
-    pause: () => {
-      const el = ref.current;
-      if (el && !lockPlay) {
-        return el.pause();
-      }
-    },
-    seek: (time: number) => {
-      const el = ref.current;
-      if (!el || (state.duration === undefined)) return;
-      time = Math.min(state.duration, Math.max(0, time));
-      el.currentTime = time;
-    },
-    volume: (volume: number) => {
-      const el = ref.current;
-      if (!el) return;
-      volume = Math.min(1, Math.max(0, volume));
-      el.volume = volume;
-      setState({volume});
-    },
-    mute: () => {
-      const el = ref.current;
-      if (!el) return;
-      el.muted = true;
-    },
-    unmute: () => {
-      const el = ref.current;
-      if (!el) return;
-      el.muted = false;
-    },
-  };
-
-  useEffect(() => {
-    const el = ref.current!;
-
-    if (!el) {
-      if (process.env.NODE_ENV !== 'production') {
-        console.error(
-          'useAudio() ref to <audio> element is empty at mount. ' +
-          'It seem you have not rendered the audio element, which is ' +
-          'returns as the first argument const [audio] = useAudio(...).'
-        );
-      }
-      return;
-    }
-
-    // Start media, if autoPlay requested.
-    if (props.autoPlay && el.paused) {
-      controls.play();
-    }
-
-    setState({
-      volume: el.volume,
-      muted: el.muted,
-    });
-  }, [props.src]);
-
-  return [element, state, controls, ref];
-};
+const useAudio = createHTMLMediaHook('audio');
 
 export default useAudio;
diff --git a/src/useVideo.ts b/src/useVideo.ts
new file mode 100644
index 00000000..11489994
--- /dev/null
+++ b/src/useVideo.ts
@@ -0,0 +1,5 @@
+import createHTMLMediaHook from './util/createHTMLMediaHook';
+
+const useVideo = createHTMLMediaHook('video');
+
+export default useVideo;
diff --git a/src/util/createHTMLMediaHook.ts b/src/util/createHTMLMediaHook.ts
new file mode 100644
index 00000000..a9073da5
--- /dev/null
+++ b/src/util/createHTMLMediaHook.ts
@@ -0,0 +1,182 @@
+import * as React from 'react';
+import {useEffect, useRef, ReactRef} from '../react';
+import useSetState from '../useSetState';
+import parseTimeRanges from './parseTimeRanges';
+
+export interface HTMLMediaProps extends React.AudioHTMLAttributes<any>, React.VideoHTMLAttributes<any> {
+  src: string;
+}
+
+export interface HTMLMediaState {
+  buffered: any[];
+  duration: number;
+  isPlaying: boolean;
+  muted: boolean;
+  time: number;
+  volume: number;
+}
+
+export interface HTMLMediaControls {
+  play: () => Promise<void> | void;
+  pause: () => void;
+  mute: () => void;
+  unmute: () => void;
+  volume: (volume: number) => void;
+  seek: (time: number) => void;
+}
+
+const createHTMLMediaHook = (tag: 'audio' | 'video') => {
+  const hook = (props: HTMLMediaProps): [React.ReactElement<HTMLMediaProps>, HTMLMediaState, HTMLMediaControls, ReactRef<HTMLAudioElement | null>] => {
+    const [state, setState] = useSetState<HTMLMediaState>({
+      buffered: [],
+      time: 0,
+      duration: 0,
+      isPlaying: false,
+      muted: false,
+      volume: 1,
+    });
+    const ref = useRef<HTMLAudioElement | null>(null);
+
+    const wrapEvent = (userEvent, proxyEvent?) => {
+      return (event) => {
+        try {
+          proxyEvent && proxyEvent(event);
+        } finally {
+          userEvent && userEvent(event);
+        }
+      };
+    };
+
+    const onPlay = () => setState({isPlaying: true});
+    const onPause = () => setState({isPlaying: false});
+    const onVolumeChange = () => {
+      const el = ref.current;
+      if (!el) return;
+      setState({
+        muted: el.muted,
+        volume: el.volume,
+      });
+    };
+    const onDurationChange = () => {
+      const el = ref.current;
+      if (!el) return;
+      const {duration, buffered} = el;
+      setState({
+        duration,
+        buffered: parseTimeRanges(buffered),
+      });
+    };
+    const onTimeUpdate = () => {
+      const el = ref.current;
+      if (!el) return;
+      setState({time: el.currentTime});
+    };
+    const onProgress = () => {
+      const el = ref.current;
+      if (!el) return;
+      setState({buffered: parseTimeRanges(el.buffered)});
+    };
+
+    const element = React.createElement(tag, {
+      controls: false,
+      ...props,
+      ref,
+      onPlay: wrapEvent(props.onPlay, onPlay),
+      onPause: wrapEvent(props.onPause, onPause),
+      onVolumeChange: wrapEvent(props.onVolumeChange, onVolumeChange),
+      onDurationChange: wrapEvent(props.onDurationChange, onDurationChange),
+      onTimeUpdate: wrapEvent(props.onTimeUpdate, onTimeUpdate),
+      onProgress: wrapEvent(props.onProgress, onProgress),
+    });
+
+    // Some browsers return `Promise` on `.play()` and may throw errors
+    // if one tries to execute another `.play()` or `.pause()` while that
+    // promise is resolving. So we prevent that with this lock.
+    // See: https://bugs.chromium.org/p/chromium/issues/detail?id=593273
+    let lockPlay: boolean = false;
+
+    const controls = {
+      play: () => {
+        const el = ref.current;
+        if (!el) return undefined;
+
+        if (!lockPlay) {
+          const promise = el.play();
+          const isPromise = typeof promise === 'object';
+
+          if (isPromise) {
+            lockPlay = true;
+            const resetLock = () => {
+              lockPlay = false;
+            };
+            promise.then(resetLock, resetLock);
+          }
+
+          return promise;
+        }
+        return undefined;
+      },
+      pause: () => {
+        const el = ref.current;
+        if (el && !lockPlay) {
+          return el.pause();
+        }
+      },
+      seek: (time: number) => {
+        const el = ref.current;
+        if (!el || (state.duration === undefined)) return;
+        time = Math.min(state.duration, Math.max(0, time));
+        el.currentTime = time;
+      },
+      volume: (volume: number) => {
+        const el = ref.current;
+        if (!el) return;
+        volume = Math.min(1, Math.max(0, volume));
+        el.volume = volume;
+        setState({volume});
+      },
+      mute: () => {
+        const el = ref.current;
+        if (!el) return;
+        el.muted = true;
+      },
+      unmute: () => {
+        const el = ref.current;
+        if (!el) return;
+        el.muted = false;
+      },
+    };
+
+    useEffect(() => {
+      const el = ref.current!;
+
+      if (!el) {
+        if (process.env.NODE_ENV !== 'production') {
+          console.error(
+            'useAudio() ref to <audio> element is empty at mount. ' +
+            'It seem you have not rendered the audio element, which is ' +
+            'returns as the first argument const [audio] = useAudio(...).'
+          );
+        }
+        return;
+      }
+
+      // Start media, if autoPlay requested.
+      if (props.autoPlay && el.paused) {
+        controls.play();
+      }
+
+      setState({
+        volume: el.volume,
+        muted: el.muted,
+      });
+    }, [props.src]);
+
+    return [element, state, controls, ref];
+  };
+
+  return hook;
+};
+
+
+export default createHTMLMediaHook;

From 8670c7480f315054cf9e04403ad6edab8642d884 Mon Sep 17 00:00:00 2001
From: streamich <streamich@gmail.com>
Date: Mon, 29 Oct 2018 21:21:55 +0100
Subject: [PATCH 2/2] =?UTF-8?q?feat:=20=F0=9F=8E=B8=20allow=20to=20pass=20?=
 =?UTF-8?q?React=20<video>=20el=20in=20useVideo=20and=20useAudio?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 docs/useAudio.md                   |  3 +-
 docs/useVideo.md                   | 12 ++++----
 src/__stories__/useVideo.story.tsx |  7 ++---
 src/util/createHTMLMediaHook.ts    | 49 ++++++++++++++++++++++--------
 4 files changed, 48 insertions(+), 23 deletions(-)

diff --git a/docs/useAudio.md b/docs/useAudio.md
index 6f74730c..9a7c374e 100644
--- a/docs/useAudio.md
+++ b/docs/useAudio.md
@@ -38,8 +38,9 @@ const Demo = () => {
 
 ## Reference
 
-```ts
+```jsx
 const [audio, state, controls, ref] = useAudio(props);
+const [audio, state, controls, ref] = useAudio(<audio {...props}/>);
 ```
 
 `audio` is React's `<audio>` element that you have to insert somewhere in your
diff --git a/docs/useVideo.md b/docs/useVideo.md
index 14f0e5c0..ab8725eb 100644
--- a/docs/useVideo.md
+++ b/docs/useVideo.md
@@ -9,10 +9,9 @@ Creates `<video>` element, tracks its state and exposes playback controls.
 import {useVideo} from 'react-use';
 
 const Demo = () => {
-  const [video, state, controls, ref] = useVideo({
-    src: 'http://clips.vorwaerts-gmbh.de/big_buck_bunny.mp4',
-    autoPlay: true,
-  });
+  const [video, state, controls, ref] = useVideo(
+    <video src="http://clips.vorwaerts-gmbh.de/big_buck_bunny.mp4" autoPlay />
+  );
 
   return (
     <div>
@@ -38,8 +37,9 @@ const Demo = () => {
 
 ## Reference
 
-```ts
-const [video, state, controls, ref] = useAudio(props);
+```jsx
+const [video, state, controls, ref] = useVideo(props);
+const [video, state, controls, ref] = useVideo(<video {...props}/>);
 ```
 
 `video` is React's `<video>` element that you have to insert somewhere in your
diff --git a/src/__stories__/useVideo.story.tsx b/src/__stories__/useVideo.story.tsx
index 1b3e3e7a..4b8b74b2 100644
--- a/src/__stories__/useVideo.story.tsx
+++ b/src/__stories__/useVideo.story.tsx
@@ -4,10 +4,9 @@ import {useVideo} from '..';
 import ShowDocs from '../util/ShowDocs';
 
 const Demo = () => {
-  const [video, state, controls, ref] = useVideo({
-    src: 'http://clips.vorwaerts-gmbh.de/big_buck_bunny.mp4',
-    autoPlay: true,
-  });
+  const [video, state, controls, ref] = useVideo(
+    <video src="http://clips.vorwaerts-gmbh.de/big_buck_bunny.mp4" autoPlay />
+  );
 
   return (
     <div>
diff --git a/src/util/createHTMLMediaHook.ts b/src/util/createHTMLMediaHook.ts
index a9073da5..1fbbf7ea 100644
--- a/src/util/createHTMLMediaHook.ts
+++ b/src/util/createHTMLMediaHook.ts
@@ -26,7 +26,17 @@ export interface HTMLMediaControls {
 }
 
 const createHTMLMediaHook = (tag: 'audio' | 'video') => {
-  const hook = (props: HTMLMediaProps): [React.ReactElement<HTMLMediaProps>, HTMLMediaState, HTMLMediaControls, ReactRef<HTMLAudioElement | null>] => {
+  const hook = (elOrProps: HTMLMediaProps | React.ReactElement<HTMLMediaProps>): [React.ReactElement<HTMLMediaProps>, HTMLMediaState, HTMLMediaControls, ReactRef<HTMLAudioElement | null>] => {
+    let element: React.ReactElement<any> | undefined;
+    let props: HTMLMediaProps;
+
+    if (React.isValidElement(elOrProps)) {
+      element = elOrProps;
+      props = element.props;
+    } else {
+      props = elOrProps as HTMLMediaProps;
+    }
+
     const [state, setState] = useSetState<HTMLMediaState>({
       buffered: [],
       time: 0,
@@ -77,17 +87,32 @@ const createHTMLMediaHook = (tag: 'audio' | 'video') => {
       setState({buffered: parseTimeRanges(el.buffered)});
     };
 
-    const element = React.createElement(tag, {
-      controls: false,
-      ...props,
-      ref,
-      onPlay: wrapEvent(props.onPlay, onPlay),
-      onPause: wrapEvent(props.onPause, onPause),
-      onVolumeChange: wrapEvent(props.onVolumeChange, onVolumeChange),
-      onDurationChange: wrapEvent(props.onDurationChange, onDurationChange),
-      onTimeUpdate: wrapEvent(props.onTimeUpdate, onTimeUpdate),
-      onProgress: wrapEvent(props.onProgress, onProgress),
-    });
+    if (element) {
+      element = React.cloneElement(element, {
+        controls: false,
+        ...props,
+        ref,
+        onPlay: wrapEvent(props.onPlay, onPlay),
+        onPause: wrapEvent(props.onPause, onPause),
+        onVolumeChange: wrapEvent(props.onVolumeChange, onVolumeChange),
+        onDurationChange: wrapEvent(props.onDurationChange, onDurationChange),
+        onTimeUpdate: wrapEvent(props.onTimeUpdate, onTimeUpdate),
+        onProgress: wrapEvent(props.onProgress, onProgress),
+      });
+    } else {
+      element = React.createElement(tag, {
+        controls: false,
+        ...props,
+        ref,
+        onPlay: wrapEvent(props.onPlay, onPlay),
+        onPause: wrapEvent(props.onPause, onPause),
+        onVolumeChange: wrapEvent(props.onVolumeChange, onVolumeChange),
+        onDurationChange: wrapEvent(props.onDurationChange, onDurationChange),
+        onTimeUpdate: wrapEvent(props.onTimeUpdate, onTimeUpdate),
+        onProgress: wrapEvent(props.onProgress, onProgress),
+      } as any); // TODO: fix this typing.
+    }
+
 
     // Some browsers return `Promise` on `.play()` and may throw errors
     // if one tries to execute another `.play()` or `.pause()` while that