Skip to content

Commit

Permalink
Proposed solution for issue videojs#656 - "Default flag not working o…
Browse files Browse the repository at this point in the history
…n tracks (subtitles, captions)"

Added additional check to look for auto preload option to avoid loading external captions, subtitles, etc. file on default
  • Loading branch information
Chris-DL committed Apr 16, 2014
1 parent 3bafdee commit 8735935
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions src/js/tracks.js
Original file line number Diff line number Diff line change
Expand Up @@ -745,6 +745,12 @@ vjs.TextTrackMenuItem = vjs.MenuItem.extend({
vjs.MenuItem.call(this, player, options);

this.player_.on(track.kind() + 'trackchange', vjs.bind(this, this.update));

// Add check for auto preload option to not load captions, subtitles, etc. on default
var preload = player.options_['preload'];
if(track.dflt() && preload === 'auto') {
this.player_.showTextTrack(this.track.id_, this.track.kind());
}
}
});

Expand Down

0 comments on commit 8735935

Please sign in to comment.