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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Card
and LegacyCard
mobile styles
#11926
Conversation
@heyjoethomas Just wanted to double check from a design standpoint that we always want to remove shadow styles when |
288afa0
to
5410382
Compare
Correct. |
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.
Looks good! Simple and effective change 馃挴
Looks good 馃憤 Would LegacyCard be updated in a separate PR? |
5410382
to
41e2eb1
Compare
Card
mobile stylesCard
and LegacyCard
mobile styles
Card
and LegacyCard
mobile stylesCard
and LegacyCard
mobile styles
Closing in favor of build phase PR |
WHY are these changes introduced?
Closes https://github.com/Shopify/polaris-internal/issues/1559
Closes https://github.com/Shopify/mobile/issues/33783
WHAT is this pull request doing?
Removes
Card
andLegacyCard
default bevel styles below thesm
breakpoint. Essentially if card corners aren鈥檛 rounded then shadow styles are removed as well.Currently the
Card
component togglesborder-radius
at certain breakpoints with theroundedAbove
prop (and defaults to thesm
breakpoint). This PR also addsbox-shadow
to these toggled styles.sm
breakpoint)sm
breakpoint)How to 馃帺
Card
Storybook BeforeCard
Storybook AfterLegacyCard
Storybook BeforeLegacyCard
Storybook After