From e17da7dfd467e5d970400a32860f6653bcc0f556 Mon Sep 17 00:00:00 2001 From: Kimberley Jensen Date: Fri, 7 Feb 2020 15:00:04 +0100 Subject: [PATCH] Bail out of focus trap if fullscreen is not active - detailed in https://github.com/sampotts/plyr/issues/1665 --- src/js/utils/elements.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/js/utils/elements.js b/src/js/utils/elements.js index 4f10938e..921d533a 100644 --- a/src/js/utils/elements.js +++ b/src/js/utils/elements.js @@ -257,10 +257,11 @@ export function trapFocus(element = null, toggle = false) { const focusable = getElements.call(this, 'button:not(:disabled), input:not(:disabled), [tabindex]'); const first = focusable[0]; const last = focusable[focusable.length - 1]; + const player = this; const trap = event => { // Bail if not tab key or not fullscreen - if (event.key !== 'Tab' || event.keyCode !== 9) { + if (event.key !== 'Tab' || event.keyCode !== 9 || !player.fullscreen.active) { return; }