From a1712be7f2c56806793a206f19e331adf0855b8a Mon Sep 17 00:00:00 2001 From: Lance Curtis <69400491+imprimisxo@users.noreply.github.com> Date: Sat, 8 Aug 2026 12:08:12 -0500 Subject: [PATCH] fix(skills): define .video-wrapper.pip-pill in talking-head-recut template pip-pill is referenced three times (GSAP lookup table rows for pip bottom-right and top-left, and the className toggle snippet) and defined zero times, so every pip following the table renders with no chrome. Not a misspelling of .framed: that rule is radius + shadow, while pip-pill is documented as radius + white ring + shadow. Renaming would silently drop the ring, so the missing rule is defined instead. The ring is a box-shadow spread rather than a border so it adds no layout box and the pixel-exact pip targets stay correct. --- skills/talking-head-recut/SKILL.md | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/skills/talking-head-recut/SKILL.md b/skills/talking-head-recut/SKILL.md index a9c797ff52..cb3068405b 100644 --- a/skills/talking-head-recut/SKILL.md +++ b/skills/talking-head-recut/SKILL.md @@ -913,6 +913,17 @@ ffmpeg -y -i "$VIDEO_PATH" -c:v libx264 -crf 18 -g 30 -keyint_min 30 \ border-radius: 16px; box-shadow: 0 12px 40px rgba(0, 0, 0, 0.35); } + + /* Pip chrome: `.framed` plus a white ring, so a pip reads as a distinct + inset against a busy background. The ring is a box-shadow spread rather + than a border, so it adds no layout box and the pixel-exact pip targets + in the GSAP lookup table stay correct. */ + .video-wrapper.pip-pill { + border-radius: 16px; + box-shadow: + 0 0 0 3px rgba(255, 255, 255, 0.9), + 0 12px 40px rgba(0, 0, 0, 0.35); + }