diff options
author | Sam Potts <sam@potts.es> | 2018-04-25 19:40:23 +1000 |
---|---|---|
committer | Sam Potts <sam@potts.es> | 2018-04-25 19:40:23 +1000 |
commit | ef8e58ede4e20611c42843312907b318bba1b2f7 (patch) | |
tree | 06a07a9208cdab06b096abe28d9616d20eaa9a62 /src/js/utils.js | |
parent | f1b275aedce897b42f025afac7a0937dc5871235 (diff) | |
download | plyr-ef8e58ede4e20611c42843312907b318bba1b2f7.tar.lz plyr-ef8e58ede4e20611c42843312907b318bba1b2f7.tar.xz plyr-ef8e58ede4e20611c42843312907b318bba1b2f7.zip |
Fix for hidden buffer and incorrect use of aria-hidden
Diffstat (limited to 'src/js/utils.js')
-rw-r--r-- | src/js/utils.js | 34 |
1 files changed, 20 insertions, 14 deletions
diff --git a/src/js/utils.js b/src/js/utils.js index 2ec57a50..41419b06 100644 --- a/src/js/utils.js +++ b/src/js/utils.js @@ -375,6 +375,25 @@ const utils = { return attributes; }, + // Toggle hidden + toggleHidden(element, hidden) { + if (!utils.is.element(element)) { + return; + } + + let hide = hidden; + + if (!utils.is.boolean(hide)) { + hide = !element.hasAttribute('hidden'); + } + + if (hide) { + element.setAttribute('hidden', ''); + } else { + element.removeAttribute('hidden'); + } + }, + // Toggle class on an element toggleClass(element, className, toggle) { if (utils.is.element(element)) { @@ -393,19 +412,6 @@ const utils = { return utils.is.element(element) && element.classList.contains(className); }, - // Toggle hidden attribute on an element - toggleHidden(element, toggle) { - if (!utils.is.element(element)) { - return; - } - - if (toggle) { - element.setAttribute('hidden', ''); - } else { - element.removeAttribute('hidden'); - } - }, - // Element matches selector matches(element, selector) { const prototype = { Element }; @@ -752,7 +758,7 @@ const utils = { return null; } - return array.reduce((prev, curr) => Math.abs(curr - value) < Math.abs(prev - value) ? curr : prev); + return array.reduce((prev, curr) => (Math.abs(curr - value) < Math.abs(prev - value) ? curr : prev)); }, // Get the provider for a given URL |