From e70209f02c3ab712972e531bbf1baed7d7f34aa1 Mon Sep 17 00:00:00 2001 From: Jason Zimdars Date: Thu, 16 Oct 2025 13:44:37 -0500 Subject: [PATCH] We can just apply it to `this.element` and adjust the CSS accordingly --- app/assets/stylesheets/card-columns.css | 2 +- .../controllers/collapsible_columns_controller.js | 12 ++---------- 2 files changed, 3 insertions(+), 11 deletions(-) diff --git a/app/assets/stylesheets/card-columns.css b/app/assets/stylesheets/card-columns.css index 2ef0d65d2..77749faa1 100644 --- a/app/assets/stylesheets/card-columns.css +++ b/app/assets/stylesheets/card-columns.css @@ -73,7 +73,7 @@ transition: var(--column-transition); /* Disable transitions during restoration */ - &.no-transitions { + .no-transitions & { transition: none; } diff --git a/app/javascript/controllers/collapsible_columns_controller.js b/app/javascript/controllers/collapsible_columns_controller.js index 6ca8ce015..fc64ba414 100644 --- a/app/javascript/controllers/collapsible_columns_controller.js +++ b/app/javascript/controllers/collapsible_columns_controller.js @@ -25,19 +25,11 @@ export default class extends Controller { } #disableTransitions() { - this.columnTargets.forEach(column => { - if (this.noTransitionsClass) { - column.classList.add(this.noTransitionsClass) - } - }) + this.element.classList.add(this.noTransitionsClass) } #enableTransitions() { - this.columnTargets.forEach(column => { - if (this.noTransitionsClass) { - column.classList.remove(this.noTransitionsClass) - } - }) + this.element.classList.remove(this.noTransitionsClass) } #toggleColumn(column) {