[theme] Remove theme.mixins.gutters

packages/material-ui/src/styles/createMixins.d.ts

@@ -3,7 +3,6 @@ import { Spacing } from './createSpacing';
 import { CSSProperties } from './withStyles';
 
 export interface Mixins {
-  gutters: (styles?: CSSProperties) => CSSProperties;
   toolbar: CSSProperties;
   // ... use interface declaration merging to add custom mixins
 }

packages/material-ui/src/styles/createMixins.js

@@ -1,34 +1,5 @@
 export default function createMixins(breakpoints, spacing, mixins) {
   return {
-    gutters: (styles = {}) => {
-      // To deprecate in v4.1
-      //       warning(
-      //         false,
-      //         [
-      //           'Material-UI: Theme.mixins.gutters() is deprecated.',
-      //           'You can use the source of the mixin directly:',
-      //           `
-      // paddingLeft: theme.spacing(2),
-      // paddingRight: theme.spacing(2),
-      // [theme.breakpoints.up('sm')]: {
-      //   paddingLeft: theme.spacing(3),
-      //   paddingRight: theme.spacing(3),
-      // },
-      // `,
-      //         ].join('\n'),
-      //       );
-
-      return {
-        paddingLeft: spacing(2),
-        paddingRight: spacing(2),
-        ...styles,
-        [breakpoints.up('sm')]: {
-          paddingLeft: spacing(3),
-          paddingRight: spacing(3),
-          ...styles[breakpoints.up('sm')],
-        },
-      };
-    },
     toolbar: {
       minHeight: 56,
       [`${breakpoints.up('xs')} and (orientation: landscape)`]: {

packages/material-ui/src/styles/createMixins.test.js

@@ -3,24 +3,12 @@ import createMixins from './createMixins';
 import createMuiTheme from './createMuiTheme';
 
 describe('createMixins', () => {
-  it('should be able to override the breakpoint', () => {
+  it('should be able add other mixins', () => {
     const theme = createMuiTheme();
-    const mixins = createMixins(theme.breakpoints, theme.spacing, {});
+    const mixins = createMixins(theme.breakpoints, theme.spacing, { test: { display: 'block' } });
 
-    const mixin = mixins.gutters({
-      display: 'flex',
-      [theme.breakpoints.up('sm')]: {
-        paddingLeft: 1,
-      },
-    });
-    expect(mixin).to.deep.equal({
-      '@media (min-width:600px)': {
-        paddingLeft: 1,
-        paddingRight: 24,
-      },
-      display: 'flex',
-      paddingLeft: 16,
-      paddingRight: 16,
+    expect(mixins.test).to.deep.equal({
+      display: 'block',
     });
   });
 });

packages/material-ui/src/styles/createMuiTheme.test.js

@@ -22,12 +22,6 @@ describe('createMuiTheme', () => {
     expect(muiTheme.transitions.duration.shorter).to.not.equal(undefined);
   });
 
-  it('should use the defined spacing for the gutters mixin', () => {
-    const spacing = 100;
-    const muiTheme = createMuiTheme({ spacing });
-    expect(muiTheme.mixins.gutters().paddingLeft).to.equal(spacing * 2);
-  });
-
   describe('shadows', () => {
     it('should provide the default array', () => {
       const muiTheme = createMuiTheme();