fix(catalog): keep the animated bar chart preview in frame - #3254
Conversation
313aa7a to
7419e47
Compare
miguel-heygen
left a comment
There was a problem hiding this comment.
Fact-checked against the live production Catalog at hyperframes.heygen.com and rendered production vs this exact head side-by-side at the same 1920×1080 viewport and settled playback state. The reported clipping/positioning defect is real, and this preview-only change fixes it without touching the installable component or animation timeline. The generated Catalog payload matches the demo source. Approved.
miguel-heygen
left a comment
There was a problem hiding this comment.
Fresh exact-head approval after the restack. Fact-checked the live production Catalog and rendered production vs this head side-by-side at the same 1920×1080 viewport and settled playback state. The reported defect is real and this preview-only change fixes it without altering the installable component or timeline. Regenerated payload is source-identical (ignoring terminal newline); registry-item lint passes.
7419e47 to
d8e5c48
Compare
What
The Animated Bar Chart Catalog preview clips the card at the top and cuts off every month label at the bottom. This keeps the full card visible throughout playback.
Fixes #3250.
Why
Catalog previews should show the component within the same 1920×1080 bounds users receive. The
2.1×wrapper scale made the demo card taller than those bounds.How
Reduced the preview-only wrapper scale to
1.6×and regenerated the Catalog payload. The installable Animated Bar Chart component and its animation timing are unchanged.Test plan
Validated with:
bun run buildbun run lint:registry-items animated-bar-chartbunx oxfmt --check registry/components/animated-bar-chart/demo.html docs/public/catalog/components/animated-bar-chart.jsonnpx hyperframes check --no-contrastagainst the standalone demoBefore and after
comparison.mp4