Restore the class-based approach

aria-expanded should go on the toggle

app/javascript/controllers/collapsible_columns_controller.js

@@ -1,26 +1,27 @@
 import { Controller } from "@hotwired/stimulus"
 
 export default class extends Controller {
+  static classes = [ "collapsed" ]
   static targets = [ "column" ]
 
   toggle(event) {
-    const clickedColumn = event.target.closest("[data-collapsible-columns-target='column']")
+    const clickedColumn = event.target.closest('[data-collapsible-columns-target="column"]')
 
     if (!clickedColumn) return
 
-    const isCurrentlyExpanded = clickedColumn.getAttribute("aria-expanded") === "true"
+    const isCurrentlyCollapsed = clickedColumn.classList.contains(this.collapsedClass)
 
     this.columnTargets.forEach(column => {
-      column.setAttribute("aria-expanded", "false")
+      column.classList.add(this.collapsedClass)
     })
 
-    if (!isCurrentlyExpanded) {
-      clickedColumn.setAttribute("aria-expanded", "true")
+    if (isCurrentlyCollapsed) {
+      clickedColumn.classList.remove(this.collapsedClass)
     }
   }
 
   preventToggle(event) {
-    if (event.detail.attributeName === "aria-expanded") {
+    if (event.detail.attributeName === "class") {
       event.preventDefault()
     }
   }