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
refactor(action): large component needs to match figma design #2786
Conversation
For the semantic PR, you'll need to get a new commit to pass that check.
|
All the tests passed this time. Thanks for letting me know I missed that colon in the commit message. |
This one is blocked by #2736 |
@caripizza created an issue aimed specifically at the font-size that this PR might address. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of small things.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry @justinhough.
Just realized we need to add margin styles for each scale when it's RTL (calcite--rtl
).
There's one style there, but it looks like it's for scale M and not specific to that scale.
@asangma Just added the update. Let me know. |
👀 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Related Issue: #2458
Summary
updated large calcite-action to match Figma for sizing and spacing from icon.