Skip to content

Commit

Permalink
<3 for the review
Browse files Browse the repository at this point in the history
  • Loading branch information
oliviertassinari committed Feb 7, 2019
1 parent b96bff7 commit d5a7a81
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion docs/src/modules/components/AppDrawer.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ const styles = theme => ({
},
title: {
color: theme.palette.text.secondary,
marginBottom: 4,
marginBottom: theme.spacing(0.5),
'&:hover': {
color: theme.palette.primary.main,
},
Expand Down
8 changes: 4 additions & 4 deletions docs/src/pages/layout/grid/grid.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,11 @@ The grid system is implemented with the `Grid` component:
## Spacing

The responsive grid focuses on consistent spacing widths, rather than column width.
Material design margins and columns follow an **8px** square baseline grid.
The spacing property is an integer in the [0, 8] interval.
Material Design margins and columns follow an **8px** square baseline grid.
The spacing property is an integer between 0 and 10 inclusive.
By default, the spacing between two grid items follows a linear function: `output(spacing) = spacing * 8px`, e.g. `spacing={2}` creates a 16px wide gap.

This output transformation function can be customized using the theme, exactly like [the system spacing](/system/spacing/#transformation).
This output transformation function can be customized [using the theme](/customization/themes/#spacing).

{{"demo": "pages/layout/grid/SpacingGrid.js"}}

Expand Down Expand Up @@ -96,7 +96,7 @@ There are 3 available workarounds:
```jsx
<body>
<div style={{ padding: 20 }}>
<Grid container spacing={4}>
<Grid container spacing={5}>
//...
</Grid>
</div>
Expand Down
2 changes: 1 addition & 1 deletion packages/material-ui/src/styles/createSpacing.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ export default function createSpacing(spacing) {
[
'Material-UI: theme.spacing.unit usage has been deprecated.',
'It will be removed in v5.',
'You can replace theme.spacing.unit * 2 with theme.spacing(2).',
'You can replace `theme.spacing.unit * 2` with `theme.spacing(2)`.',
].join('\n'),
);
warnOnce = true;
Expand Down

0 comments on commit d5a7a81

Please sign in to comment.