LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

revert autoplay fix

Browse source
This commit is contained in:
Sean Yesmunt 2019-12-09 10:34:22 -05:00
parent 6deb77bccf
commit 2bc4ca0c22
1 changed files with 1 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
ui/component/viewers/videoViewer/view.jsx
View file

@ -22,6 +22,7 @@ const SEEK_STEP = 10; // time to seek in seconds
const VIDEO_JS_OPTIONS: { poster?: string } = { const VIDEO_JS_OPTIONS: { poster?: string } = {
controls: true, controls: true,
autoplay: true,
preload: 'auto', preload: 'auto',
playbackRates: [0.25, 0.5, 0.75, 1, 1.1, 1.25, 1.5, 2], playbackRates: [0.25, 0.5, 0.75, 1, 1.1, 1.25, 1.5, 2],
responsive: true, responsive: true,
@ -113,12 +114,6 @@ function VideoViewer(props: Props) {
player = videojs(videoNode, videoJsOptions, function() { player = videojs(videoNode, videoJsOptions, function() {
player.volume(volume); player.volume(volume);
player.muted(muted); player.muted(muted);
player.ready(() => {
// In the future this should be replaced with something that checks if the
// video is actually playing after calling play()
// If it's not, fall back to the play button
player.play();
});
}); });
} }
@ -133,7 +128,6 @@ function VideoViewer(props: Props) {
// requireRedraw just makes it so the video component is removed from the page _by react_ // requireRedraw just makes it so the video component is removed from the page _by react_
// Then it's set to false immediately after so we can re-mount a new player // Then it's set to false immediately after so we can re-mount a new player
setRequireRedraw(true); setRequireRedraw(true);
player.pause();
}; };
}, [videoRef, source, contentType, setRequireRedraw, requireRedraw]); }, [videoRef, source, contentType, setRequireRedraw, requireRedraw]);