ProximaEntertainment/plyr
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #379 from magourex/master

Browse Source 
Support WebVTT with CRLF or LF Line separator with firefox #378
This commit is contained in:
Sam Potts 2016-10-01 22:10:35 +10:00 committed by GitHub
commit 8098be6f93
1 changed files with 14 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/js/plyr.js
View File

@ -1026,14 +1026,25 @@
caption,
req = xhr.responseText;
captions = req.split('\n\n');
//According to webvtt spec, line terminator consists of one of the following
// CRLF (U+000D U+000A), LF (U+000A) or CR (U+000D)
var lineSeparator = '\r\n';
if(req.indexOf(lineSeparator+lineSeparator) === -1) {
if(req.indexOf('\r\r') !== -1){
lineSeparator = '\r';
} else {
lineSeparator = '\n';
}
}
captions = req.split(lineSeparator+lineSeparator);
for (var r = 0; r < captions.length; r++) {
caption = captions[r];
plyr.captions[r] = [];
// Get the parts of the captions
var parts = caption.split('\n'),
var parts = caption.split(lineSeparator),
index = 0;
// Incase caption numbers are added