From 52f883be4e5bffef05c1104dfaec2c6f5a995da9 Mon Sep 17 00:00:00 2001 From: DispatchCommit Date: Mon, 25 Jan 2021 15:50:11 -0800 Subject: [PATCH] fix linting errors --- .../viewers/videoViewer/internal/videojs.jsx | 20 +++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/ui/component/viewers/videoViewer/internal/videojs.jsx b/ui/component/viewers/videoViewer/internal/videojs.jsx index ec24b8063..65f8806f5 100644 --- a/ui/component/viewers/videoViewer/internal/videojs.jsx +++ b/ui/component/viewers/videoViewer/internal/videojs.jsx @@ -354,7 +354,7 @@ export default React.memo(function VideoJs(props: Props) { if (!player) { return; } - + // Add various event listeners to player player.one('play', onInitialPlay); player.on('volumechange', onVolumeChange); @@ -363,7 +363,7 @@ export default React.memo(function VideoJs(props: Props) { // Replace volume bar with custom LBRY volume bar LbryVolumeBarClass.replaceExisting(player); - + // initialize mobile UI player.mobileUi(); // Inits mobile version. No-op if Desktop. @@ -376,9 +376,6 @@ export default React.memo(function VideoJs(props: Props) { displayCurrentQuality: true, }); - // Add reference to player to global scope - window.player = player; - // fixes #3498 (https://github.com/lbryio/lbry-desktop/issues/3498) // summary: on firefox the focus would stick to the fullscreen button which caused buggy behavior with spacebar player.on('fullscreenchange', () => document.activeElement && document.activeElement.blur()); @@ -391,6 +388,9 @@ export default React.memo(function VideoJs(props: Props) { const vjsElement = createVideoPlayerDOM(containerRef.current); const vjsPlayer = initializeVideoPlayer(vjsElement); + // Add reference to player to global scope + window.player = vjsPlayer; + // Add event listener for keyboard shortcuts window.addEventListener('keydown', handleKeyDown); @@ -402,7 +402,7 @@ export default React.memo(function VideoJs(props: Props) { player.dispose(); window.player = undefined; } - } + }; }, []); // Update video player and reload when source URL changes @@ -419,9 +419,9 @@ export default React.memo(function VideoJs(props: Props) { // override type if we receive an .m3u8 (transcoded mp4) if ( - response && - response.redirected && - response.url && + response && + response.redirected && + response.url && response.url.endsWith('m3u8') ) { type = 'application/x-mpegURL'; @@ -429,7 +429,7 @@ export default React.memo(function VideoJs(props: Props) { // Update player poster // note: the poster prop seems to return null usually. - if ( poster ) player.poster(poster); + if (poster) player.poster(poster); // Update player source player.src({