feat: set label for vttPlaylist from label when available (#161)

Co-authored-by: Roman Pylypets <roman.pylypets@lamin.ar>
videojs · Oct 17, 2022 · 3fc0486 · 3fc0486
1 parent d1aeee9
commit 3fc0486
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/src/toM3u8.js b/src/toM3u8.js
@@ -217,7 +217,7 @@ export const organizeAudioPlaylists = (playlists, sidxMapping = {}, isAudioOnly
 
 export const organizeVttPlaylists = (playlists, sidxMapping = {}) => {
  return playlists.reduce((a, playlist) => {
- const label = playlist.attributes.lang || 'text';
+ const label = playlist.attributes.label || playlist.attributes.lang || 'text';
 
  if (!a[label]) {
  a[label] = {

diff --git a/test/toM3u8.test.js b/test/toM3u8.test.js
@@ -911,10 +911,27 @@ QUnit.test('playlists with label', function(assert) {
  type: 'static'
  },
  segments: []
+ }, {
+ attributes: {
+ sourceDuration: 100,
+ id: '1',
+ width: 800,
+ height: 600,
+ codecs: 'foo;bar',
+ duration: 0,
+ bandwidth: 10000,
+ periodStart: 0,
+ mimeType: 'text/vtt',
+ type: 'static',
+ label
+ },
+ segments: []
  }];
  const output = toM3u8({ dashPlaylists });
 
  assert.ok(label in output.mediaGroups.AUDIO.audio, 'label exists');
+ assert.ok(label in output.mediaGroups.SUBTITLES.subs, 'label exists');
+
 });
 
 QUnit.test('608 captions', function(assert) {