Skip to content

feat: HMZE podcast reference — footer badge + DE About video embed#446

Merged
rdmueller merged 2 commits into
LLM-Coding:mainfrom
raifdmueller:add-hmze-podcast-reference
Apr 21, 2026
Merged

feat: HMZE podcast reference — footer badge + DE About video embed#446
rdmueller merged 2 commits into
LLM-Coding:mainfrom
raifdmueller:add-hmze-podcast-reference

Conversation

@raifdmueller

@raifdmueller raifdmueller commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a reference to the HMZE podcast episode #58 (Beyond Vibe Coding — Semantic Anchors als Magic Spells) where Ralf explained the concept in detail. Two touchpoints:

  • Footer badge on both EN and DE — "As seen on / Zu Gast bei HMZE (DE)" with the podcast logo, linking to https://www.youtube.com/watch?v=rQj-B3VTx48.
  • DE About page only — click-to-load video embed above the first section. The English About page is intentionally left unchanged; a 60-minute German explainer would be the wrong primary intro for an English-speaking reader.

DSGVO / privacy

The embed is implemented as a click-to-load facade:

  • Before click: only the static thumbnail from i.ytimg.com loads. No cookies, no YouTube JS, no tracking.
  • After click: iframe swaps in with youtube-nocookie.com (extended privacy endpoint), autoplay via user gesture.

Verified in preview: network panel shows only the thumbnail request until the button is clicked; click then loads the no-cookie iframe.

Assets

website/public/hmze-logo.png — 114×96 PNG derived from the HMZE channel logo at hmze.tech/assets/img/logo.jpeg. The asset is used in a press-mention context for a confirmed appearance; can be swapped out later if the podcast provides an official press asset.

Also

Regenerates docs/all-anchors.adoc, website/public/llms.txt, and website/public/sitemap.xml to include the four anchors merged in #444 and #445 (SLAP, Occam's Razor, Code Smells, WWCND) — the previous PRs missed this regeneration.

Test plan

  • Unit tests pass (89/89)
  • Preview build verified in Chrome: thumbnail + play button + title overlay render on DE About page
  • Click-to-load works: iframe swaps in, src is youtube-nocookie.com
  • No YouTube network requests before click (only i.ytimg.com for thumbnail)
  • Footer badge shows on both EN and DE
  • EN About page has no video embed (as designed)
  • Logo asset added to .gitignore allowlist

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

  • New Features

    • YouTube-Video-Embeds mit Lazy-Loading-Funktionalität hinzugefügt – Videos werden nur bei Klick geladen
    • HMZE-Podcast-Referenz in der Fußzeile ergänzt
  • Documentation

    • Fünf neue Semantic-Anchor-Dokumente hinzugefügt: Code Smells, Single Level of Abstraction Principle, Occam's Razor, What Would Chuck Norris Do und Lehman's Software Classification

raifdmueller and others added 2 commits April 21, 2026 09:44
- Footer: new "As seen on / Zu Gast bei" row with HMZE logo linking to
  https://www.youtube.com/watch?v=rQj-B3VTx48; shown on both languages,
  labelled "(DE)" since the podcast is German.
- DE About page: click-to-load YouTube facade above "Was sind Semantic
  Anchors?". Only the static thumbnail loads until the user clicks;
  after activation the iframe uses youtube-nocookie.com. EN About stays
  unchanged — a 60-minute German explainer does not belong as the
  primary intro on the English page.
- youtube-facade.js utility: small hydrator scanning doc content for
  button.youtube-facade[data-video-id]. Keeps the embed DSGVO-safe
  without consent banners.
- New i18n keys: footer.asSeenOn, footer.hmzeAlt.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Picks up the 4 anchors merged in LLM-Coding#444 and LLM-Coding#445 that the previous PRs
did not regenerate (SLAP, Occam's Razor, Code Smells, WWCND).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

Diese PR fügt eine YouTube-Facade-Funktion hinzu, die es ermöglicht, YouTube-Videos auf Klick zu laden. Die Änderungen umfassen ein neues Utility-Modul, Komponenten-Anpassungen, CSS-Styling, Dokumentation in AsciiDoc-Format und Lokalisierungsschlüssel in Deutsch und Englisch.

Changes

Cohort / Datei(en) Zusammenfassung
Feature-Implementierung
website/src/utils/youtube-facade.js, website/src/components/doc-page.js, website/src/components/footer.js, website/src/styles/main.css
Neues YouTube-Facade-Utility mit Click-to-Load-Funktionalität; Integration in doc-page für Dokumentations-Inhalte; HMZE-Logo-Link mit Styling in Footer; neue CSS-Klassen für Facade-Wrapper, Hover-Effekte und Fade-Out-Gradient.
Dokumentation & Inhalte
docs/about.de.adoc, docs/all-anchors.adoc, website/public/llms.txt
YouTube-Embed-Block in About-Seite; neue AsciiDoc-Includes für semantische Anker (Code Smells, SLAP, Occam's Razor, WWCND, Lehman's Classification); aktualisierte Catalog-Einträge in llms.txt.
Lokalisierung
website/src/translations/de.json, website/src/translations/en.json
Neue Footer-Übersetzungsschlüssel für "As seen on" / "Zu Gast bei" und HMZE-Podcast-Beschreibung in Deutsch und Englisch.
Konfiguration & Metadaten
.gitignore, website/public/data/metadata.json, website/public/sitemap.xml
Exception für hmze-logo.png in .gitignore; aktualisierter Zeitstempel in metadata.json; Sitemap mit neuen Anker-URLs und aktualisiertem lastmod-Datum.

Sequence Diagram(s)

sequenceDiagram
    participant User as Benutzer
    participant DocPage as doc-page.js
    participant Facade as youtube-facade.js
    participant DOM as DOM/Browser
    participant YouTube as YouTube (nocookie)

    User->>DocPage: Dokumentseite laden
    DocPage->>DOM: Dokumenten-HTML laden
    DocPage->>Facade: hydrateYouTubeFacades(contentEl) aufrufen
    Facade->>DOM: Alle youtube-facade Buttons finden
    Facade->>DOM: Click-Handler registrieren
    Facade->>DOM: data-hydrated="true" setzen
    Note over Facade: Wartet auf Benutzer-Interaktion
    
    User->>DOM: YouTube-Facade-Button klicken
    DOM->>Facade: Click-Handler ausgelöst
    Facade->>Facade: Video-ID validieren
    Facade->>DOM: Facade-Button durch iframe ersetzen
    Facade->>DOM: youtube-loaded Klasse setzen
    DOM->>YouTube: iframe lädt YouTube-Video
    YouTube->>DOM: Video wird angezeigt
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • PR #111: Die Änderungen an docs/all-anchors.adoc und website/public/llms.txt überschneiden sich direkt mit den generierten Inhalten und Metadaten der neuen semantischen Anker aus PR #111.
  • PR #112: Beide PRs modifizieren die gleiche Footer-Komponente (website/src/components/footer.js) und fügen Footer-Übersetzungsschlüssel hinzu (website/src/translations/*.json).
  • PR #384: Beide PRs nehmen überlappende Änderungen an website/public/sitemap.xml vor — aktualisieren <lastmod>-Werte und fügen neue Anker-URL-Einträge hinzu.
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 25.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Der PR-Titel beschreibt präzise die Hauptänderungen: Hinzufügen eines HMZE-Podcast-Verweises mit Footer-Badge und YouTube-Video-Embed auf der deutschen About-Seite.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@rdmueller rdmueller merged commit d80c9d7 into LLM-Coding:main Apr 21, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants