Changeset fa08981 in Klonkt for src/assets/js/audio-player.js

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@…>

File:

• src/assets/js/audio-player.js (modified) (2 diffs)

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',