flowplayer / flowplayer-hlsjs

Flowplayer HLS.js plugin
MIT License
81 stars 35 forks source link

if master manifest has name, show it as label #109

Closed mysamimi closed 6 years ago

mysamimi commented 6 years ago

my change is .... if (level.name) { label = level.name; } else if (level.width && level.height) { ... VS code beautify another lines :)

bbbo commented 6 years ago

@nnarhinen perhaps an option for hlsjs-lite as well?

mysamimi commented 6 years ago

hlsjs already extract NAME from master manifest, but flow player-hlsjs ignore it. this coed use master manifest label.

nnarhinen commented 6 years ago

We will not maintain this plugin anymore. You should start using the internal hlsjs engine of flowplayer.

If you still want this functionality, you should port your changes to that repository. Also FYI this pull request contains too much unrelated whitespace changes to get merged.

mysamimi commented 6 years ago

tnx Niklas