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

apply starting position directly (no setTimeout)

Browse source
This commit is contained in:
Travis Eden 2017-12-21 20:22:40 -05:00
parent 6c4a4e593c
commit 7d29c4b91d
1 changed files with 1 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/renderer/component/video/internal/player.jsx
View file

@ -38,11 +38,6 @@ class VideoPlayer extends React.PureComponent {
position, position,
} = this.props; } = this.props;
// I'm using a Timeout because I'm not sure where this should happen
if (position > 0) {
setTimeout(() => (mediaElement.currentTime = position), 900);
}
const loadedMetadata = e => { const loadedMetadata = e => {
this.setState({ hasMetadata: true, startedPlaying: true }); this.setState({ hasMetadata: true, startedPlaying: true });
this.refs.media.children[0].play(); this.refs.media.children[0].play();
@ -75,6 +70,7 @@ class VideoPlayer extends React.PureComponent {
document.addEventListener("keydown", this.togglePlayListener); document.addEventListener("keydown", this.togglePlayListener);
const mediaElement = this.refs.media.children[0]; const mediaElement = this.refs.media.children[0];
if (mediaElement) { if (mediaElement) {
mediaElement.currentTime = position;
mediaElement.addEventListener("play", () => this.props.doPlay()); mediaElement.addEventListener("play", () => this.props.doPlay());
mediaElement.addEventListener("pause", () => { mediaElement.addEventListener("pause", () => {
console.log("CURRENT TIME:", mediaElement.currentTime); console.log("CURRENT TIME:", mediaElement.currentTime);