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

hack-y solution to losing player context

Browse source 
use a global window variable to store the reference to the current video.js instance.
This commit is contained in:
DispatchCommit 2021-01-23 12:35:43 -08:00 committed by Sean Yesmunt
parent 68ae2d571f
commit d8564cda5a
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
ui/component/viewers/videoViewer/internal/videojs.jsx
View file

@ -170,7 +170,7 @@ export default React.memo<Props>(function VideoJs(props: Props) {
type: sourceType,
},
],
autoplay: false,
autoplay: true,
poster: poster, // thumb looks bad in app, and if autoplay, flashing poster is annoying
plugins: {
eventTracking: true,
@ -414,6 +414,8 @@ export default React.memo<Props>(function VideoJs(props: Props) {
useEffect(() => {
// For some reason the video player is responsible for detecting content type this way
fetch(source, { method: 'HEAD' }).then(response => {
const player = window.player;
if (!player) {
console.log(`Our player was disposed, we should disregard the fetch result.`);
return;