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

additional tweaks to renderAudio

Browse source
This commit is contained in:
Akinwale Ariwodola 2017-07-06 20:03:31 +01:00
parent 913d642b52
commit 95145b57e3
1 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
ui/js/component/video/internal/player.jsx
View file

@ -22,7 +22,6 @@ class VideoPlayer extends React.PureComponent {
}
componentDidMount() {
const component = this;
const container = this.refs.media;
const { contentType, downloadPath, mediaType } = this.props;
const loadedMetadata = e => {
@ -44,7 +43,7 @@ class VideoPlayer extends React.PureComponent {
// use renderAudio override for mp3
if (VideoPlayer.MP3_CONTENT_TYPES.indexOf(contentType) > -1) {
this.renderAudio(container, false);
this.renderAudio(container, null, false);
} else {
player.append(
this.file(),
@ -85,6 +84,11 @@ class VideoPlayer extends React.PureComponent {
}
renderAudio(container, autoplay) {
if (container.firstChild) {
container.firstChild.remove();
}
// clear the container
const { downloadPath } = this.props;
const audio = document.createElement("audio");
audio.autoplay = autoplay;
@ -125,7 +129,7 @@ class VideoPlayer extends React.PureComponent {
const container = this.refs.media.children[0];
if (VideoPlayer.MP3_CONTENT_TYPES.indexOf(contentType) > -1) {
this.renderAudio(container, true);
this.renderAudio(this.refs.media, true);
} else {
player.render(this.file(), container, {
autoplay: true,