1
0
mirror of https://github.com/videojs/video.js.git synced 2025-02-18 12:24:02 +02:00

Merge pull request #1075 from seniorflexdeveloper/hotfix/spinner

Loading Spinner Fixes
This commit is contained in:
Matthew McClure 2014-03-13 14:26:20 -07:00
commit 80d7b5014a

View File

@ -15,8 +15,6 @@ vjs.LoadingSpinner = vjs.Component.extend({
player.on('canplay', vjs.bind(this, this.hide));
player.on('canplaythrough', vjs.bind(this, this.hide));
player.on('playing', vjs.bind(this, this.hide));
player.on('seeked', vjs.bind(this, this.hide));
player.on('seeking', vjs.bind(this, this.show));
// in some browsers seeking does not trigger the 'playing' event,
@ -25,6 +23,7 @@ vjs.LoadingSpinner = vjs.Component.extend({
player.on('seeked', vjs.bind(this, this.hide));
player.on('error', vjs.bind(this, this.show));
player.on('ended', vjs.bind(this, this.hide));
// Not showing spinner on stalled any more. Browsers may stall and then not trigger any events that would remove the spinner.
// Checked in Chrome 16 and Safari 5.1.2. http://help.videojs.com/discussions/problems/883-why-is-the-download-progress-showing