diff options
author | Robin Appelman <robin@icewind.nl> | 2016-07-27 17:06:20 +0300 |
---|---|---|
committer | Robin Appelman <robin@icewind.nl> | 2016-07-27 17:21:34 +0300 |
commit | 77c215226cbb05e81866239913a43fb2d1b6f8b6 (patch) | |
tree | 363dfa4910be47a230a194095c10ecef0eb4ab98 /js/viewer.js | |
parent | 18d6667710f1f9586fff578660d064ee44a60deb (diff) |
better videojs lifecicle management
Diffstat (limited to 'js/viewer.js')
-rwxr-xr-x | js/viewer.js | 23 |
1 files changed, 11 insertions, 12 deletions
diff --git a/js/viewer.js b/js/viewer.js index 7727f3e..eeb71a2 100755 --- a/js/viewer.js +++ b/js/viewer.js @@ -33,13 +33,16 @@ var videoViewer = { // show elements $('#videoplayer_overlay').fadeIn('fast'); // initialize player - var vjsPlayer = videojs("my_video_1"); - // append close button to video element - $("#my_video_1").append('<a class="icon-view-close" id="box-close" href="#"></a>'); - // add event to close button - $('#box-close').click(videoViewer.hidePlayer); - // autoplay - vjsPlayer.play(); + videojs("my_video_1").ready(function() { + videoViewer.player = this; + // append close button to video element + $("#my_video_1").append('<a class="icon-view-close" id="box-close" href="#"></a>'); + // add event to close button + $('#box-close').click(videoViewer.hidePlayer); + // autoplay + videoViewer.player.play(); + }); + }, hide : function() { $('#videoplayer_overlay').fadeOut('fast', function() { @@ -76,13 +79,9 @@ var videoViewer = { }); }, hidePlayer : function() { + videoViewer.player.dispose(); videoViewer.player = false; - delete videoViewer.player; videoViewer.UI.hide(); - // force close socket - $('video').each(function() { - $($(this)[0]).attr('src', ''); - }); }, log : function(message){ console.log(message); |