diff options
author | Guru Prasad Srinivasa <gurupras@buffalo.edu> | 2016-02-16 19:53:07 -0500 |
---|---|---|
committer | Guru Prasad Srinivasa <gurupras@buffalo.edu> | 2016-02-16 20:17:10 -0500 |
commit | 402c45ee2bf1d528ea1c3ea333dfa7fcb7dd79af (patch) | |
tree | 0a9ae7a4e033e00d071e8b88879f3ab92238aed4 /src/js | |
parent | 67f19166ac92d5adc03c9421300149260a6424ac (diff) | |
download | plyr-402c45ee2bf1d528ea1c3ea333dfa7fcb7dd79af.tar.lz plyr-402c45ee2bf1d528ea1c3ea333dfa7fcb7dd79af.tar.xz plyr-402c45ee2bf1d528ea1c3ea333dfa7fcb7dd79af.zip |
Updated manual caption split pattern
Previously, Plyr was using a fixed pattern of '\n\n' to split
contents into captions. This does not always work as some VTT files
contain '\r\n'. This commit checks for both.
Diffstat (limited to 'src/js')
-rw-r--r-- | src/js/plyr.js | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/src/js/plyr.js b/src/js/plyr.js index 2f50bb69..b2b9c964 100644 --- a/src/js/plyr.js +++ b/src/js/plyr.js @@ -1523,12 +1523,18 @@ record, req = xhr.responseText; - records = req.split('\n\n'); - + var pattern = '\n'; + records = req.split(pattern + pattern); + if(records.length === 1) { + // The '\n' pattern didn't work + // Try '\r\n' + pattern = '\r\n'; + records = req.split(pattern + pattern); + } for (var r = 0; r < records.length; r++) { record = records[r]; plyr.captions[r] = []; - plyr.captions[r] = record.split('\n'); + plyr.captions[r] = record.split(pattern); } // Remove first element ('VTT') |