Skip to content

Commit

Permalink
Card padding in mobile is now 16px instead of 20px
Browse files Browse the repository at this point in the history
  • Loading branch information
amireynoso committed Nov 13, 2018
1 parent a8a8854 commit 8f950e1
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 5 deletions.
1 change: 1 addition & 0 deletions UNRELEASED.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ Use [the changelog guidelines](https://git.io/polaris-changelog-guidelines) to f
### Bug fixes

- Fixed `Autocomplete` empty state example Markdown not parsing correctly ([#592](https://github.com/Shopify/polaris-react/pull/592))
- Fixed `Card` spacing in small devices ([#608](https://github.com/shopify/polaris-react/pull/608))

### Documentation

Expand Down
27 changes: 22 additions & 5 deletions src/components/Card/Card.scss
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,19 @@
}

.Header {
padding: spacing(loose) spacing(loose) 0;
padding: spacing() spacing() 0;

@include page-content-when-not-fully-condensed {
padding: spacing(loose) spacing(loose) 0;
}
}

.Section {
padding: spacing(loose);
padding: spacing();

@include page-content-when-not-fully-condensed {
padding: spacing(loose);
}

+ .Section {
border-top: border-width() solid color('sky');
Expand All @@ -40,13 +48,22 @@
padding-bottom: spacing(tight);

.Section-fullWidth & {
padding-left: spacing(loose);
padding-right: spacing(loose);
padding-left: spacing();
padding-right: spacing();

@include page-content-when-not-fully-condensed {
padding-left: spacing(loose);
padding-right: spacing(loose);
}
}
}

.Footer {
display: flex;
justify-content: flex-end;
padding: 0 spacing(loose) spacing(loose);
padding: 0 spacing() spacing();

@include page-content-when-not-fully-condensed {
padding: 0 spacing(loose) spacing(loose);
}
}

0 comments on commit 8f950e1

Please sign in to comment.