You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Would expect plyr instances to be destroyed after calling instances.forEach(function(instance) { instance.destroy(); }
Actual behaviour
Unforunately the js instances are still there, the method only destroys the DOM elements. Because of the instance still existing, the entire plyr DOM elements can still be found as detached elements in the memory.
Environment
Browser: Chrome
Version: latest
Operating System: Windows
Version: latest
Players affected:
HTML5 Video
HTML5 Audio
YouTube
Vimeo
Steps to reproduce
Instantiate player(s), call destroy() method on it and then try and access the instance. For example call the play() method on it. Even though the initial DOM elements are destroyed, you're still able to play the audio file...
@sampotts it seems that this issue is with plyr.container not getting garbage collected. I've created a PR for this, could you please assign someone for review and get it merged if it's fine? We really need this as well for our project.
Expected behaviour
Would expect plyr instances to be destroyed after calling
instances.forEach(function(instance) { instance.destroy(); }
Actual behaviour
Unforunately the js instances are still there, the method only destroys the DOM elements. Because of the instance still existing, the entire plyr DOM elements can still be found as detached elements in the memory.
Environment
Players affected:
Steps to reproduce
Instantiate player(s), call
destroy()
method on it and then try and access the instance. For example call theplay()
method on it. Even though the initial DOM elements are destroyed, you're still able to play the audio file...Relevant links
#77
The text was updated successfully, but these errors were encountered: