Pass multiple classes to `classList.add` / `classList.remove` (#30537)

This is supported by our currently supported browsers.

diff --git a/js/src/carousel.js b/js/src/carousel.js
index 3020974d382b0882ea50ee150cb1aba42d718733..a5451538cde511603f74e8e96a1431eae5abb49f 100644 (file)
--- a/js/src/carousel.js
+++ b/js/src/carousel.js
@@ -478,13 +478,10 @@ class Carousel {
 
       EventHandler
         .one(activeElement, TRANSITION_END, () => {
-          nextElement.classList.remove(directionalClassName)
-          nextElement.classList.remove(orderClassName)
+          nextElement.classList.remove(directionalClassName, orderClassName)
           nextElement.classList.add(CLASS_NAME_ACTIVE)
 
-          activeElement.classList.remove(CLASS_NAME_ACTIVE)
-          activeElement.classList.remove(orderClassName)
-          activeElement.classList.remove(directionalClassName)
+          activeElement.classList.remove(CLASS_NAME_ACTIVE, orderClassName, directionalClassName)
 
           this._isSliding = false
 

diff --git a/js/src/collapse.js b/js/src/collapse.js
index fde3e0a9ed826d1e0fedd85d028aa95813e137aa..4529a85ec36105c507cd8b8b4d1563c4d0e41d8b 100644 (file)
--- a/js/src/collapse.js
+++ b/js/src/collapse.js
@@ -192,8 +192,7 @@ class Collapse {
 
     const complete = () => {
       this._element.classList.remove(CLASS_NAME_COLLAPSING)
-      this._element.classList.add(CLASS_NAME_COLLAPSE)
-      this._element.classList.add(CLASS_NAME_SHOW)
+      this._element.classList.add(CLASS_NAME_COLLAPSE, CLASS_NAME_SHOW)
 
       this._element.style[dimension] = ''
 
@@ -230,8 +229,7 @@ class Collapse {
     reflow(this._element)
 
     this._element.classList.add(CLASS_NAME_COLLAPSING)
-    this._element.classList.remove(CLASS_NAME_COLLAPSE)
-    this._element.classList.remove(CLASS_NAME_SHOW)
+    this._element.classList.remove(CLASS_NAME_COLLAPSE, CLASS_NAME_SHOW)
 
     const triggerArrayLength = this._triggerArray.length
     if (triggerArrayLength > 0) {

diff --git a/js/src/popover.js b/js/src/popover.js
index 2d423f0dd54ded7326b8e89fe065e1a6c44f0e1d..4aac7e8ab2d5504184fa6688641800fe98bbe910 100644 (file)
--- a/js/src/popover.js
+++ b/js/src/popover.js
@@ -113,8 +113,7 @@ class Popover extends Tooltip {
 
     this.setElementContent(SelectorEngine.findOne(SELECTOR_CONTENT, tip), content)
 
-    tip.classList.remove(CLASS_NAME_FADE)
-    tip.classList.remove(CLASS_NAME_SHOW)
+    tip.classList.remove(CLASS_NAME_FADE, CLASS_NAME_SHOW)
   }
 
   _addAttachmentClass(attachment) {

diff --git a/js/src/tooltip.js b/js/src/tooltip.js
index 9a1dff3d6e1dfffe154c023d2ad52dc3867ae6f0..3f40b3da12a44a3101f79b5678f78fc620d30ab1 100644 (file)
--- a/js/src/tooltip.js
+++ b/js/src/tooltip.js
@@ -400,8 +400,7 @@ class Tooltip {
   setContent() {
     const tip = this.getTipElement()
     this.setElementContent(SelectorEngine.findOne(SELECTOR_TOOLTIP_INNER, tip), this.getTitle())
-    tip.classList.remove(CLASS_NAME_FADE)
-    tip.classList.remove(CLASS_NAME_SHOW)
+    tip.classList.remove(CLASS_NAME_FADE, CLASS_NAME_SHOW)
   }
 
   setElementContent(element, content) {