refactor(admin): simplify css and js of card collapsing and side bar directories

1 files changed, 12 insertions, 16 deletions

diff --git a/internal/admin/static/itemeditor.js b/internal/admin/static/itemeditor.js
index 2283c91..4097e64 100644
--- a/internal/admin/static/itemeditor.js
+++ b/internal/admin/static/itemeditor.js
@@ -216,6 +216,9 @@ function renderItemEditor(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();
   setupCraftTypeChange();
   setupEquipTypeChange();
@@ -238,20 +241,19 @@ 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">';
+  h += '</summary>';
+  h += '<div class="obj-card-body">';
 
     if (sec.isArray) {
       if (sec.interactionStyle) {
@@ -315,8 +317,7 @@ function renderCard(sec, data) {
       }
     }
     h += '</div>';
-  }
-  h += '</div>';
+  h += '</details>';
   return h;
 }
 
@@ -1031,11 +1032,6 @@ function renderKV(sec, il, data, keyOptions) {
 // Card toggle / section add-remove
 // =========================================================================
 
-function toggleCard(id) {
-  expandedCards[id] = expandedCards[id] === false ? true : false;
-  renderCurrent();
-}
-
 function toggleCardWidth(id) {
   cardWidths[id] = !cardWidths[id];
   renderCurrent();