Changeset 5f47671 in Klonkt

Timestamp:

06/20/2026 02:37:34 AM (3 months ago)

Author:

roboburr <roboburr@…>

Branches:

main

Children:

16b0c00

Parents:

1720482

Message:

fix: tablet opens full-player + clean search snippet

• audio-player: the mini-player click only jumps to the post on true desktop (≥1200, no full-player). Tablet + phone now open the full-player (sheet), like mobile. (Was: ≥768 counted as desktop → tablet jumped to the post.) buster audio-player.js?v=20.
• search: FTS snippet is cleaned up — shortcodes ([[playlist:…]]) and markdown removed, <mark> highlights kept; falls back to the excerpt if nothing readable remains.

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

Location:

src

Files:

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

src/assets/js/audio-player.js

@@ -605 +605 @@
   expandTrigger.addEventListener('click', () => {
     const t = queue[currentIndex];
-    const isDesktop = window.matchMedia && window.matchMedia('(min-width: 768px)').matches;
-    if (isDesktop && t) {
+    // Alleen op échte desktop (≥1200, geen full-player) springen we naar de post.
+    // Tablet + telefoon hebben een full-player → open die (zoals mobiel).
+    const jumpToPost = !hasFullPlayer();
+    if (jumpToPost && t) {
       // 1) Track speelde vanuit een post → die URL kennen we al.
       if (t.postUrl) { goToPost(t.postUrl, t.id); return; }

src/routes/search.js

@@ -37 +37 @@
 function likeArg(q) {
   return '%' + q.replace(/[%_\\]/g, '\\$&') + '%';
+}
+
+// Maak de FTS-snippet leesbaar: shortcodes ([[...]]) en markdown-tekens eruit.
+// De <mark>…</mark>-highlights van FTS blijven staan. Valt terug op de excerpt
+// (geëscaped) als er na het schoonmaken niets leesbaars overblijft.
+function cleanSnippet(html, excerpt) {
+  const esc = (s) => s.replace(/&/g, '&amp;').replace(/</g, '&lt;').replace(/>/g, '&gt;');
+  const s = (html || '')
+    .replace(/!\[[^\]]*\]\([^)]*\)/g, ' ')   // markdown-afbeeldingen
+    .replace(/\[\[[^\]]*?\]\]/g, ' ')         // shortcodes (incl. <mark> erin)
+    .replace(/\[\[|\]\]/g, ' ')               // losse haakjes door truncatie
+    .replace(/[#>*_`~]+/g, ' ')               // markdown-opmaaktekens
+    .replace(/\s+/g, ' ')
+    .trim();
+  if (!s || /^[…\s]*$/.test(s)) return esc((excerpt || '').slice(0, 160));
+  return s;
 }
 
@@ -85 +101 @@
         LIMIT 50
       `).all(ftsQuery, site.id);
+      results = results.map((r) => ({ ...r, snippet: cleanSnippet(r.snippet, r.excerpt) }));
     } catch (err) {
       queryError = err.message;

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=19"></script>
+<script src="/assets/js/audio-player.js?v=20"></script>
 <!-- Eigen custom media-embeds (YouTube/SoundCloud/Spotify) via de echte
      player-API's + gedeelde mutual-exclusion registry met de site-speler. -->