-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[feat] Inline promo #566
Comments
Questions:
|
@hellogreg @markcaron, since we're still working on documenting correct usage and finding out whether this can extend card accessibly, can this be moved to the Chansey milestone? |
@marionnegp Done! |
@markcaron I'm looking into this component now and Marion asked some great questions.
Looking at Inline Promo, do you think it should be its own component or should it be an |
FWIW I'm leaning towards card variant. Can we maybe put together a PoC and shop it around to studio devs? |
Description
The inline promo needs to be formalized as a component because it's being used consistently on .com and will be used as a personalization.
Acceptance Criteria
Image
Featured inline promo:
Standard inline promo:
Full-width promo:
Link to design doc
Figma file
Other resources
No response
The text was updated successfully, but these errors were encountered: