Skip to content

Fix podcast cover layout regression (gap, clipped pill, missing date/logo) - #442

Merged
alexeygrigorev merged 1 commit into
mainfrom
hub/intake-5edf27d40d5aabecf92ca75b4c77f2f571ea94b7237ce48dcc7333504fcfd72a
Nov 13, 2026
Merged

alexeygrigorev merged 1 commit into
mainfrom
hub/intake-5edf27d40d5aabecf92ca75b4c77f2f571ea94b7237ce48dcc7333504fcfd72a

Conversation

@alexeygrigorev

Copy link
Copy Markdown
Member

Fixes intake b0213fc8 (Madhu square 2000x2000 podcast cover).

Root cause: deployed bundle e8f5c854… one-speaker podcast templates used absolute positions (title top 360, portrait top 1050 square); short titles left a huge gap, portrait+pill sat against the bottom edge, and the template had no date/time or DataTalks.Club logo fields. Known-good reference: original Figma-derived kit in dataops-knowledge banner-generator/templates/podcast-1-speaker-luma / youtube-podcast (title directly above large portrait, pill inside canvas).

Fix: flow-layout templates (podcastLayout.ts), eventDate/eventTime/eventWhen from Card eventTime/anchorDate, new bundle 09191882… (uploaded, content-addressed) incl. logo asset, bundle key parameter bumped.

Tests: backend/e2e/banner-podcast-layout.spec.ts (square+landscape x short/normal/long titles: bboxes inside canvas, logo/date present, gap bound), plus unit test for schedule fields.

Not touching Luma. Existing non-podcast-1 templates unchanged.

…ks.Club logo

Root cause: the deployed bundle's podcast-1 templates pinned the title at a fixed top
and the portrait at a fixed y (square: top 1050), so short titles left a huge void and
the portrait + name pill sat at the bottom edge; no date/time or logo fields existed.
Compared against the original Figma-derived kit (dataops-knowledge banner-generator):
centered title directly above a large portrait, pill inside canvas.

Adds eventDate/eventTime/eventWhen fields from the Card, a flow-layout template
generator, a bundle build script, bundle key bump, docs and a Chromium geometry
regression spec for square and landscape.
@alexeygrigorev
alexeygrigorev merged commit ff779a5 into main Nov 13, 2026
5 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.

1 participant