fix loadedmetada disabled

[SzHeJason]

No description provided.

[vhmth]

Nice! Does .ready guarantee the duration information has been loaded? My guess is that @spchuang had this listening to loadedmetadata for that reason.

[spchuang]

@SzHeJason can you explain more what issue does using 'player.ready' solve?

[SzHeJason]

Sorry,My english is poor . Can't listen LoadedMetaData event when using this plugin. I think it should not ban any native event.

[SzHeJason]

player.ready is working in my project , Maybe it's a bad fixed

[spchuang]

can you try something like this..

player.ready(function(){
    if (player.readyState && player.readyState() > 0) {
      initialize();
    } else {
      player.on("loadedmetadata", function() {
        initialize();
      });
    }
  });

And what browser are you on? And what video (flash, html5) are you playing?

Thanks

[SzHeJason]

If register loadedmetadata event to setup plugin, you must turn off loadedmetadata event in player.Still can‘t listen loadedmetadata in player

Video : HTML5
System : macOS
Browser : Chrome 63.0.3239.84

[monsieurludo]

Off loadedmetadata event listener should just be tied to initialize function to avoid issues with playlists handling where loadedmetada must refresh cues each time on loadedmetadata

[monsieurludo]

This is what needs to be done to isolate the off loadedmetadata event listener to any other loadedmetadata listeners:
function initialize() {
if (setting.markerTip.display) {
initializeMarkerTip();
}

    // remove existing markers if already initialized
    player.markers.removeAll();
    addMarkers(setting.markers);

    onTimeUpdate();
    player.on("timeupdate", onTimeUpdate);
    player.off("loadedmetadata", initialize);
  }

  // setup the plugin after we loaded video's meta data
  player.on("loadedmetadata", initialize);

[monsieurludo]

Playlist management by detaching loadedmetadata to any other loadedmetadata event listeners