[TablePagination] Remove select icon override

mui · Jan 27, 2020 · a477ec8 · a477ec8
1 parent 143e908
commit a477ec8
Show file tree

Hide file tree

Showing 7 changed files with 9 additions and 10 deletions.
diff --git a/packages/material-ui/src/Chip/Chip.js b/packages/material-ui/src/Chip/Chip.js
@@ -167,7 +167,7 @@ export const styles = theme => {
         backgroundColor: fade(theme.palette.secondary.main, theme.palette.action.hoverOpacity),
       },
     },
-    // TODO remove in V5
+    // TODO v5: remove
     /* Styles applied to the `avatar` element. */
     avatar: {},
     /* Styles applied to the `avatar` element if `size="small"`. */

diff --git a/packages/material-ui/src/MenuItem/MenuItem.js b/packages/material-ui/src/MenuItem/MenuItem.js
@@ -19,7 +19,7 @@ export const styles = theme => ({
       minHeight: 'auto',
     },
   },
-  // TODO To remove in v5?
+  // TODO v5: remove
   /* Styles applied to the root element if `disableGutters={false}`. */
   gutters: {},
   /* Styles applied to the root element if `selected={true}`. */

diff --git a/packages/material-ui/src/Slider/Slider.js b/packages/material-ui/src/Slider/Slider.js
@@ -159,7 +159,7 @@ export const styles = theme => ({
   },
   /* Styles applied to the root element if `color="primary"`. */
   colorPrimary: {
-    // TODO v5, move the style here
+    // TODO v5: move the style here
   },
   /* Styles applied to the root element if `color="secondary"`. */
   colorSecondary: {
@@ -277,7 +277,7 @@ export const styles = theme => ({
   },
   /* Styles applied to the thumb element if `color="primary"`. */
   thumbColorPrimary: {
-    // TODO v5, move the style here
+    // TODO v5: move the style here
   },
   /* Styles applied to the thumb element if `color="secondary"`. */
   thumbColorSecondary: {

diff --git a/packages/material-ui/src/TablePagination/TablePagination.js b/packages/material-ui/src/TablePagination/TablePagination.js
@@ -48,10 +48,9 @@ export const styles = theme => ({
     textAlign: 'right',
     textAlignLast: 'right', // Align <select> on Chrome.
   },
+  // TODO v5: remove
   /* Styles applied to the Select component `icon` class. */
-  selectIcon: {
-    top: 1,
-  },
+  selectIcon: {},
   /* Styles applied to the `InputBase` component. */
   input: {
     color: 'inherit',

diff --git a/packages/material-ui/src/styles/createTypography.js b/packages/material-ui/src/styles/createTypography.js
@@ -78,7 +78,7 @@ export default function createTypography(palette, typography) {
     {
       htmlFontSize,
       pxToRem,
-      round, // TODO To remove in v5?
+      round, // TODO v5: remove
       fontFamily,
       fontSize,
       fontWeightLight,

diff --git a/packages/material-ui/src/useMediaQuery/useMediaQueryTheme.js b/packages/material-ui/src/useMediaQuery/useMediaQueryTheme.js
@@ -1,6 +1,6 @@
 import useMediaQuery from './useMediaQuery';
 
-// TODO to deprecate in v4.x and remove in v5
+// TODO v5: to deprecate in v4.x and remove in v5
 export default function useMediaQueryTheme(...args) {
   return useMediaQuery(...args);
 }
diff --git a/packages/material-ui/src/utils/setRef.js b/packages/material-ui/src/utils/setRef.js
@@ -1,4 +1,4 @@
-// TODO: Make it private only in v5
+// TODO v5: make it private
 export default function setRef(ref, value) {
   if (typeof ref === 'function') {
     ref(value);