Runtime error when using multiple plyr instances where one was destroyed

Run https://jsfiddle.net/jason_sanjose/s0c1g4vz/ and follow instructions in comments.

Summary:

• Multiple plyr instances in use

• Call destroy() on any one instance

• Toggle fullscreen on a remaining plyr instance

• Observe runtime error due to orphaned event handler from destroyed instance (see below)

• Issue does not already exist

• Issue observed on https://plyr.io

Expected behaviour

No error

Actual behaviour

plyr.js:1 Uncaught TypeError: Cannot read property 'querySelectorAll' of null
    at B (plyr.js:1)
    at J (plyr.js:1)
    at HTMLDocument.he (plyr.js:1)

Environment

• Browser: Chrome
• Version: 63
• Operating System: Windows
• Version: 10

Players affected:

• HTML5 Video
• HTML5 Audio
• YouTube
• Vimeo

Steps to reproduce

Run https://jsfiddle.net/jason_sanjose/s0c1g4vz/ and follow instructions in comments

Relevant links

Need a matching _off for https://github.com/sampotts/plyr/blob/master/src/js/plyr.js#L3198 when destroying a plyr instance

[sampotts]

This should be fixed in 3.0.0-beta.18 - I can now call destroy() and get no errors. Sorry about that…

[sampotts]

Closing this one as I think it’s fixed but feel free to re-open if not solved in v3 👍