From 4db6bf7a2e2256c20c885a536fadd4a5f4433469 Mon Sep 17 00:00:00 2001 From: Albin Larsson Date: Wed, 9 May 2018 22:08:09 +0200 Subject: Make utils.toggleClass() compatible with Element.classList.toggle (rename toggle argument to 'force' and make it optional) --- src/js/utils.js | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) (limited to 'src/js/utils.js') diff --git a/src/js/utils.js b/src/js/utils.js index 5c4934a1..0a1c8fcc 100644 --- a/src/js/utils.js +++ b/src/js/utils.js @@ -390,14 +390,16 @@ const utils = { } }, - // Toggle class on an element - toggleClass(element, className, toggle) { + // Mirror Element.classList.toggle, with IE compatibility for "force" argument + toggleClass(element, className, force) { if (utils.is.element(element)) { - const contains = element.classList.contains(className); - - element.classList[toggle ? 'add' : 'remove'](className); + let method = 'toggle'; + if (typeof force !== 'undefined') { + method = force ? 'add' : 'remove'; + } - return (toggle && !contains) || (!toggle && contains); + element.classList[method](className); + return element.classList.contains(className); } return null; -- cgit v1.2.3