Merge pull request #20654 from Snuffleupagus/eslint-unicorn-rules

Enable a couple of additional `eslint-plugin-unicorn` rules

Author: timvandermeij

eslint.config.mjs

@@ -139,6 +139,7 @@ export default [
       "unicorn/no-unnecessary-array-flat-depth": "error",
       "unicorn/no-unnecessary-array-splice-count": "error",
       "unicorn/no-unnecessary-slice-end": "error",
+      "unicorn/no-useless-collection-argument": "error",
       "unicorn/no-useless-promise-resolve-reject": "error",
       "unicorn/no-useless-spread": "error",
       "unicorn/prefer-array-find": "error",
@@ -147,6 +148,7 @@ export default [
       "unicorn/prefer-array-index-of": "error",
       "unicorn/prefer-array-some": "error",
       "unicorn/prefer-at": "error",
+      "unicorn/prefer-classlist-toggle": "error",
       "unicorn/prefer-date-now": "error",
       "unicorn/prefer-dom-node-append": "error",
       "unicorn/prefer-dom-node-remove": "error",

web/debugger.mjs

@@ -333,11 +333,10 @@ class Stepper {
     });
 
     showBoxesCheckbox.addEventListener("change", () => {
-      if (showBoxesCheckbox.checked) {
-        this.pageContainer.classList.add("showDebugBoxes");
-      } else {
-        this.pageContainer.classList.remove("showDebugBoxes");
-      }
+      this.pageContainer.classList.toggle(
+        "showDebugBoxes",
+        showBoxesCheckbox.checked
+      );
     });
   }