From 223d198c95b64d3d803be5d914c5b899dd30c8eb Mon Sep 17 00:00:00 2001
From: historia <[not public]>
Date: Mon, 13 Jul 2026 19:59:34 -0400
Subject: refactor(admin): simplify css and js of card collapsing and side bar
 directories

---
 internal/admin/static/hazardeditor.js | 48 ++++++++++++++++-------------------
 1 file changed, 22 insertions(+), 26 deletions(-)

(limited to 'internal/admin/static/hazardeditor.js')

diff --git a/internal/admin/static/hazardeditor.js b/internal/admin/static/hazardeditor.js
index 1f5fdf3..f85cde3 100644
--- a/internal/admin/static/hazardeditor.js
+++ b/internal/admin/static/hazardeditor.js
@@ -114,6 +114,9 @@ function renderHazardEditor(id, data) {
   html += '</div>';
 
   $('#editorMain').innerHTML = html;
+  document.querySelectorAll('details.obj-card').forEach(function(d) {
+    d.addEventListener('toggle', function() { expandedCards[d.getAttribute('data-card')] = d.open; });
+  });
   setupSearchFields();
 }
 
@@ -124,35 +127,33 @@ function renderCurrent() {
 }
 
 function renderCard(sec, data) {
-  var collapsed = expandedCards[sec.id] === false;
+  var openAttr = expandedCards[sec.id] === false ? '' : ' open';
   var isFull = cardWidths[sec.id] !== undefined ? cardWidths[sec.id] : (sec.fullWidth || false);
   var fullClass = isFull ? ' obj-card-full' : '';
-  var h = '<div class="obj-card' + fullClass + '" data-card="' + sec.id + '">';
-  h += '<div class="obj-card-header" onclick="toggleCard(\'' + sec.id + '\')">';
-  h += '<span class="obj-card-arrow">' + (collapsed ? '&#9654;' : '&#9660;') + '</span>';
+  var h = '<details class="obj-card' + fullClass + '" data-card="' + sec.id + '"' + openAttr + '>';
+  h += '<summary class="obj-card-header">';
+  h += '<span class="obj-card-arrow"></span>';
   h += '<span class="obj-card-label">' + sec.label + '</span>';
-  h += '<button class="btn btn-sm obj-card-expand" onclick="event.stopPropagation();toggleCardWidth(\'' + sec.id + '\')" title="' + (isFull ? 'Half width' : 'Full width') + '">' + (isFull ? '\u2190' : '\u2192') + '</button>';
+  h += '<button class="btn btn-sm obj-card-expand" onclick="event.preventDefault();toggleCardWidth(\'' + sec.id + '\')" title="' + (isFull ? 'Half width' : 'Full width') + '">' + (isFull ? '\u2190' : '\u2192') + '</button>';
   if (!sec.always) {
-    h += '<button class="btn btn-sm btn-danger obj-card-remove" onclick="event.stopPropagation();removeSection(\'' + sec.id + '\')">X</button>';
+    h += '<button class="btn btn-sm btn-danger obj-card-remove" onclick="event.preventDefault();removeSection(\'' + sec.id + '\')">X</button>';
   }
-  h += '</div>';
-  if (!collapsed) {
-    h += '<div class="obj-card-body">';
-    if (sec.id === 'combat') {
-      h += renderHazardCombatCard(data);
-    } else {
-      h += '<div class="obj-card-grid">';
-      sec.fields.forEach(function(f) {
-        h += renderField(sec, f, data, -1);
-      });
-      h += '</div>';
-      if (sec.id === 'core') {
-        h += ced_renderTagHelp('color');
-      }
-    }
+  h += '</summary>';
+  h += '<div class="obj-card-body">';
+  if (sec.id === 'combat') {
+    h += renderHazardCombatCard(data);
+  } else {
+    h += '<div class="obj-card-grid">';
+    sec.fields.forEach(function(f) {
+      h += renderField(sec, f, data, -1);
+    });
     h += '</div>';
+    if (sec.id === 'core') {
+      h += ced_renderTagHelp('color');
+    }
   }
   h += '</div>';
+  h += '</details>';
   return h;
 }
 
@@ -160,11 +161,6 @@ function renderField(sec, f, data, idx) {
   return ced_renderField(sec, f, data, idx, null, null, cardPath, fieldID, getVal);
 }
 
-function toggleCard(id) {
-  expandedCards[id] = expandedCards[id] === false ? true : false;
-  renderCurrent();
-}
-
 function toggleCardWidth(id) {
   cardWidths[id] = !cardWidths[id];
   renderCurrent();
-- 
cgit v1.2.3