feat(lynx): add AspectRatio component#1648
Conversation
🦋 Changeset detectedLatest commit: 6cc0adc The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
df48c6f to
c317571
Compare
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
6cc0adc to
c113c93
Compare
Summary
Adds
AspectRatioto@seed-design/lynx-react.Changes
AspectRatiouses the nativeaspect-ratioCSS property (no::before/padding-bottom hack — Lynx has no pseudo-element styling).overflowdefaults tohidden(matches web). radius/stroke are left to ImageFrame.examples/lynx-spa(VariantCatalog withpicsumplaceholder images,mode="aspectFill")flex flex-colwas missing on the wrapper)Verification
Stack
PR 4/4 — base:
lynx-toggle-pkg(PR #1647). AspectRatio itself is independent; it sits at the end of the stack. Merge #1645 → #1646 → #1647 → this.🤖 Generated with Claude Code