Changeset 650b601 in Klonkt

Timestamp:

06/15/2026 02:02:56 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

4c9f29a

Parents:

d352b63

Message:

feat: embed pause applies to every embed (not just .folio-embed)

Broaden the blur handler that pauses the site player: now triggers on focus
to any iframe instead of only iframes inside .folio-embed. On the site,
iframes are by definition embeds (YouTube/Spotify/SoundCloud/Vimeo/Apple Music/
Bandcamp), so this covers all platforms without gaps. Cache-buster v=12 -> v=13.

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

Location:

src

Files:

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

src/assets/js/audio-player.js

@@ -547 +547 @@
   });
 
-  // Geen dubbel geluid: zodra de gebruiker een ingesloten video/audio aanklikt
-  // (YouTube/Vimeo/SoundCloud/Spotify-iframe in een .folio-embed) gaat de focus
-  // naar dat iframe -> window 'blur'. Speelt onze speler dan? Pauzeer 'm.
+  // Geen dubbel geluid: élke embed (YouTube/Vimeo/SoundCloud/Spotify/Apple Music/
+  // Bandcamp) is een iframe. Zodra de gebruiker er een aanklikt om af te spelen,
+  // gaat de focus naar dat iframe -> window 'blur'. Speelt onze speler dan?
+  // Pauzeer 'm. (Op de site zijn iframes per definitie embeds.)
   window.addEventListener('blur', () => {
     setTimeout(() => {
       const el = document.activeElement;
-      if (el && el.tagName === 'IFRAME' && el.closest('.folio-embed') && audio.src && !audio.paused) {
+      if (el && el.tagName === 'IFRAME' && audio.src && !audio.paused) {
         pause();
       }

src/views/shell.ejs

@@ -309 +309 @@
      ?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=12"></script>
+<script src="/assets/js/audio-player.js?v=13"></script>
 <% if (site && site.enable_audio_player && audioTracks && audioTracks.length > 0) { %>
   <script>window.PCMS_SITE_TRACKS = <%- JSON.stringify(audioTracks) %>;</script>