Changeset fa08981 in Klonkt

Timestamp:

06/20/2026 01:52:46 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

8f6225c

Parents:

230e446

Message:

fix(player): scroll-to-track works for site-wide tracks too

Site-wide tracks (PCMS_SITE_TRACKS) were not getting an id → no anchor; and
no postUrl → no jump. Now: id included in the queue; mini-player click uses
postUrl when known, otherwise looks up the post via new endpoint
GET /audio/track/:id/post (newest post with [[track:<id>]]) and jumps +
scrolls to the track. audio-player.js?v=18.

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

Location:

src

Files:

• assets/js/audio-player.js (modified) (2 diffs)
• routes/audio.js (modified) (1 diff)
• views/shell.ejs (modified) (1 diff)

src/assets/js/audio-player.js

@@ -592 +592 @@
     const t = queue[currentIndex];
     const isDesktop = window.matchMedia && window.matchMedia('(min-width: 768px)').matches;
-    if (isDesktop && t && t.postUrl) { goToPost(t.postUrl, t.id); return; }
+    if (isDesktop && t) {
+      // 1) Track speelde vanuit een post → die URL kennen we al.
+      if (t.postUrl) { goToPost(t.postUrl, t.id); return; }
+      // 2) Site-brede track (geen postUrl) → zoek de post op via de track-id.
+      if (t.id) {
+        fetch('/audio/track/' + encodeURIComponent(t.id) + '/post')
+          .then((r) => (r.ok ? r.json() : null))
+          .then((d) => { if (d && d.url) goToPost(d.url, t.id); else openSheet(); })
+          .catch(() => openSheet());
+        return;
+      }
+    }
     openSheet();
   });
@@ -831 +842 @@
     if (Array.isArray(window.PCMS_SITE_TRACKS) && window.PCMS_SITE_TRACKS.length) {
       queue = window.PCMS_SITE_TRACKS.map(t => ({
+        id:     t.id || null,
         url:    t.media_url || t.url,
         title:  t.title  || 'Untitled',

src/routes/audio.js

@@ -141 +141 @@
 });
 
+// Welke post bevat deze track? (voor de mini-speler → "spring naar de post +
+// scroll naar de track".) Pakt de nieuwste gepubliceerde post met [[track:<id>]].
+router.get('/track/:id/post', (req, res) => {
+  const id = String(req.params.id || '');
+  if (!/^[A-Za-z0-9_-]+$/.test(id)) return res.status(400).json({ error: 'bad id' });
+  const isHub = res.locals.tenancy === 'hub';
+  const row = db.prepare(`
+    SELECT p.slug, s.slug AS site_slug
+    FROM posts p JOIN sites s ON s.id = p.site_id
+    WHERE p.status = 'published' AND p.content LIKE ?
+    ORDER BY p.published_at DESC LIMIT 1
+  `).get('%[[track:' + id + ']]%');
+  if (!row) return res.status(404).json({ error: 'not found' });
+  const url = isHub ? `/user/${row.site_slug}/${row.slug}` : `/${row.slug}`;
+  res.json({ url });
+});
+
 export default router;

src/views/shell.ejs

@@ -300 +300 @@
      ?v=N — cache-buster: bump bij elke audio-player.js wijziging zodat
      Cloudflare (max-age=1y) niet de oude versie blijft serveren. -->
-<script src="/assets/js/audio-player.js?v=17"></script>
+<script src="/assets/js/audio-player.js?v=18"></script>
 <!-- Eigen custom media-embeds (YouTube/SoundCloud/Spotify) via de echte
      player-API's + gedeelde mutual-exclusion registry met de site-speler. -->