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
fix(block): Corrects alignment of slotted icon #6883
Conversation
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.
🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳
🔳🔲🔳🔳🔳🔲🔳🔳🔲🔲🔳🔳🔲🔲🔲🔳🔳🔲🔲🔲🔳🔲🔲🔲🔲🔳🔲🔳
🔳🔲🔲🔳🔳🔲🔳🔲🔳🔳🔲🔳🔳🔲🔳🔳🔲🔳🔳🔳🔳🔲🔳🔳🔳🔳🔲🔳
🔳🔲🔳🔲🔳🔲🔳🔲🔳🔳🔲🔳🔳🔲🔳🔳🔲🔳🔳🔳🔳🔲🔲🔲🔳🔳🔲🔳
🔳🔲🔳🔳🔲🔲🔳🔲🔳🔳🔲🔳🔳🔲🔳🔳🔲🔳🔳🔳🔳🔲🔳🔳🔳🔳🔳🔳
🔳🔲🔳🔳🔳🔲🔳🔳🔲🔲🔳🔳🔲🔲🔲🔳🔳🔲🔲🔲🔳🔲🔲🔲🔲🔳🔲🔳
🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳🔳
@@ -43,7 +43,7 @@ | |||
<div class="child right-aligned-text">default (non-collapsible + no block-section)</div> | |||
|
|||
<div class="child"> | |||
<calcite-block heading="Fruit" summary="It's nature's candy"> </calcite-block> | |||
<calcite-block heading="Fruit" description="It's nature's candy"> </calcite-block> |
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.
Nice! ✨:broom::sparkles:
Related Issue: #6627
Summary
Adjust slotted icon container css to better align the icon container. Small misalignments due to icon may still occur but this should be better than before. Adds a handful of test-only stories to capture more Block states, and fixes some instance of an outdated prop used in demo pages.
cc @ashetland @SkyeSeitz - I think this was already correct in Figma, but FYI as this will likely require Chromatic approval.
cc @asangma @mitc7862 - FYI
Before:
After: