Changeset 084d1c0 in Klonkt

Timestamp:

06/30/2026 02:10:36 AM (2 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

e0a1ec1

Parents:

f2eacca

Message:

fix(admin-audio): fediverse-share toggle is now AJAX (no page jump)

The new per-track fedi-share toggle used a form POST + redirect, so clicking it reloaded the
page and jumped to the top. It now toggles via AJAX in place, mirroring the download toggle.

• src/views/pages/admin-audio.ejs — fedi button -> data-track-fedi + an AJAX click handler
• src/routes/admin-audio.js — the /api/:id field-update route now accepts fedi_open

Co-Authored-By: Claude <noreply@…>

Location:

src

Files:

• routes/admin-audio.js (modified) (1 diff)
• views/pages/admin-audio.ejs (modified) (2 diffs)

src/routes/admin-audio.js

@@ -494 +494 @@
     fields.push('downloadable = ?'); values.push(body.downloadable ? 1 : 0);
   }
+  if (Object.prototype.hasOwnProperty.call(body, 'fedi_open')) {
+    fields.push('fedi_open = ?'); values.push(body.fedi_open ? 1 : 0);
+  }
   if (Object.prototype.hasOwnProperty.call(body, 'credit')) {
     fields.push('credit = ?'); values.push(String(body.credit || '').trim() || null);

src/views/pages/admin-audio.ejs

@@ -123 +123 @@
                         style="<%= t.downloadable ? 'color:var(--accent,#6b8f71)' : 'opacity:.5' %>">⬇</button>
               <% } %>
-              <form action="/admin/audio/<%= t.id %>/fedi-open" method="post" class="ax-track-fedi" style="display:inline">
-                <button type="submit" class="ax-icon-btn"
-                        aria-label="<%= t.fedi_open ? _aaudFediOn : _aaudFediOff %>" title="<%= t.fedi_open ? _aaudFediOn : _aaudFediOff %>"
-                        style="<%= t.fedi_open ? 'color:var(--accent,#6b8f71)' : 'opacity:.5' %>">🌐</button>
-              </form>
+              <button type="button" class="ax-icon-btn" data-track-fedi data-id="<%= t.id %>" data-on="<%= t.fedi_open ? '1' : '0' %>"
+                      aria-label="<%= t.fedi_open ? _aaudFediOn : _aaudFediOff %>" title="<%= t.fedi_open ? _aaudFediOn : _aaudFediOff %>"
+                      style="<%= t.fedi_open ? 'color:var(--accent,#6b8f71)' : 'opacity:.5' %>">🌐</button>
               <form action="/admin/audio/<%= t.id %>/delete" method="post" data-confirm="<%= _aaudDelConfirm %>" class="ax-track-delete">
                 <button type="submit" class="ax-icon-btn ax-icon-btn-danger" aria-label="<%= _aaudDelete %>" title="<%= _aaudDelete %>">🗑</button>
@@ -1011 +1009 @@
     });
   });
+
+  // Fediverse-share (fedi_open) toggle — AJAX, no page reload (mirrors the download toggle).
+  document.querySelectorAll('[data-track-fedi]').forEach(btn => {
+    btn.addEventListener('click', async () => {
+      if (btn.disabled) return;
+      const want = btn.dataset.on === '1' ? 0 : 1;
+      btn.disabled = true;
+      try {
+        const res = await fetch('/admin/audio/api/' + btn.dataset.id, {
+          method: 'POST',
+          headers: { 'Content-Type': 'application/json' },
+          body: JSON.stringify({ fedi_open: !!want }),
+        });
+        if (!res.ok) throw new Error('HTTP ' + res.status);
+        btn.dataset.on = String(want);
+        btn.style.color = want ? 'var(--accent,#6b8f71)' : '';
+        btn.style.opacity = want ? '1' : '.5';
+        btn.title = want ? '<%= t('aaud.fedi_on') %>' : '<%= t('aaud.fedi_off') %>';
+      } catch (e) {
+        alert('<%= t('aaud.change_failed') %>: ' + (e.message || e));
+      } finally {
+        btn.disabled = false;
+      }
+    });
+  });
 })();
 </script>