[core] Avoid over-transpiling

mui#20896 (comment)
oliviertassinari · May 10, 2020 · 303263d · 303263d
1 parent a528ab4
commit 303263d
Show file tree

Hide file tree

Showing 11 changed files with 17 additions and 19 deletions.
diff --git a/docs/src/modules/constants.js b/docs/src/modules/constants.js
@@ -15,7 +15,7 @@ const LANGUAGES = ['en', 'zh', 'ru', 'pt', 'es', 'fr', 'de', 'ja', 'aa'];
 const LANGUAGES_SSR = ['en', 'zh', 'ru', 'pt', 'es'];
 
 // Work in progress
-const LANGUAGES_IN_PROGRESS = [...LANGUAGES];
+const LANGUAGES_IN_PROGRESS = LANGUAGES.slice();
 
 // Valid languages to use in production
 const LANGUAGES_LABEL = [

diff --git a/docs/src/pages/components/lists/CheckboxList.js b/docs/src/pages/components/lists/CheckboxList.js
@@ -23,7 +23,7 @@ export default function CheckboxList() {
 
   const handleToggle = (value) => () => {
     const currentIndex = checked.indexOf(value);
-    const newChecked = [...checked];
+    const newChecked = checked.slice();
 
     if (currentIndex === -1) {
       newChecked.push(value);

diff --git a/packages/material-ui-lab/src/ToggleButtonGroup/ToggleButtonGroup.js b/packages/material-ui-lab/src/ToggleButtonGroup/ToggleButtonGroup.js
@@ -58,10 +58,10 @@ const ToggleButtonGroup = React.forwardRef(function ToggleButton(props, ref) {
     let newValue;
 
     if (value && index >= 0) {
-      newValue = [...value];
+      newValue = value.slice();
       newValue.splice(index, 1);
     } else {
-      newValue = value ? [...value, buttonValue] : [buttonValue];
+      newValue = value ? value.concat(buttonValue) : [buttonValue];
     }
 
     onChange(event, newValue);

diff --git a/packages/material-ui-lab/src/TreeView/TreeView.js b/packages/material-ui-lab/src/TreeView/TreeView.js
@@ -217,7 +217,7 @@ const TreeView = React.forwardRef(function TreeView(props, ref) {
       const topLevelNodes = nodeMap.current[-1].children;
       diff = topLevelNodes.filter((node) => !isExpanded(node));
     }
-    const newExpanded = [...expanded, ...diff];
+    const newExpanded = expanded.concat(diff);
 
     if (diff.length > 0) {
       setExpandedState(newExpanded);

diff --git a/packages/material-ui-lab/src/useAutocomplete/useAutocomplete.js b/packages/material-ui-lab/src/useAutocomplete/useAutocomplete.js
@@ -500,7 +500,7 @@ export default function useAutocomplete(props) {
     let newValue = option;
 
     if (multiple) {
-      newValue = Array.isArray(value) ? [...value] : [];
+      newValue = Array.isArray(value) ? value.slice() : [];
 
       if (process.env.NODE_ENV !== 'production') {
         const matches = newValue.filter((val) => getOptionSelected(option, val));
@@ -716,7 +716,7 @@ export default function useAutocomplete(props) {
       case 'Backspace':
         if (multiple && inputValue === '' && value.length > 0) {
           const index = focusedTag === -1 ? value.length - 1 : focusedTag;
-          const newValue = [...value];
+          const newValue = value.slice();
           newValue.splice(index, 1);
           handleValue(event, newValue, 'remove-option', {
             option: value[index],
@@ -805,7 +805,7 @@ export default function useAutocomplete(props) {
   };
 
   const handleTagDelete = (index) => (event) => {
-    const newValue = [...value];
+    const newValue = value.slice();
     newValue.splice(index, 1);
     handleValue(event, newValue, 'remove-option', {
       option: value[index],

diff --git a/packages/material-ui/src/MobileStepper/MobileStepper.js b/packages/material-ui/src/MobileStepper/MobileStepper.js
@@ -88,7 +88,7 @@ const MobileStepper = React.forwardRef(function MobileStepper(props, ref) {
 
       {variant === 'dots' && (
         <div className={classes.dots}>
-          {[...new Array(steps)].map((_, index) => (
+          {Array.apply(null, Array(steps)).map((_, index) => (
             <div
               key={index}
               className={clsx(classes.dot, {

diff --git a/packages/material-ui/src/Select/SelectInput.js b/packages/material-ui/src/Select/SelectInput.js
@@ -149,7 +149,7 @@ const SelectInput = React.forwardRef(function SelectInput(props, ref) {
     let newValue;
 
     if (multiple) {
-      newValue = Array.isArray(value) ? [...value] : [];
+      newValue = Array.isArray(value) ? value.slice() : [];
       const itemIndex = value.indexOf(child.props.value);
       if (itemIndex === -1) {
         newValue.push(child.props.value);

diff --git a/packages/material-ui/src/Slider/Slider.js b/packages/material-ui/src/Slider/Slider.js
@@ -90,7 +90,7 @@ function setValueIndex({ values, source, newValue, index }) {
     return source;
   }
 
-  const output = [...values];
+  const output = values.slice();
   output[index] = newValue;
   return output;
 }
@@ -386,11 +386,12 @@ const Slider = React.forwardRef(function Slider(props, ref) {
 
   const range = Array.isArray(valueDerived);
   const instanceRef = React.useRef();
-  let values = range ? [...valueDerived].sort(asc) : [valueDerived];
+  let values = range ? valueDerived.slice().sort(asc) : [valueDerived];
   values = values.map((value) => clamp(value, min, max));
   const marks =
     marksProp === true && step !== null
-      ? [...Array(Math.floor((max - min) / step) + 1)].map((_, index) => ({
+      ? // eslint-disable-next-line prefer-spread
+        Array.apply(null, Array(Math.floor((max - min) / step) + 1)).map((_, index) => ({
           value: min + step * index,
         }))
       : marksProp || [];

diff --git a/packages/material-ui/src/test-utils/describeConformance.js b/packages/material-ui/src/test-utils/describeConformance.js
@@ -47,9 +47,7 @@ function testClassName(element, getOptions) {
 
     const wrapper = mount(React.cloneElement(element, { className }));
 
-    expect(
-      findOutermostIntrinsic(wrapper).hasClass(className)
-    ).to.equal(
+    expect(findOutermostIntrinsic(wrapper).hasClass(className)).to.equal(
       true,
       'does have a custom `className`',
     );

diff --git a/packages/material-ui/src/withWidth/withWidth.js b/packages/material-ui/src/withWidth/withWidth.js
@@ -51,7 +51,7 @@ const withWidth = (options = {}) => (Component) => {
      *            |-------|-------|-------|-------|------>
      * width      |  xs   |  sm   |  md   |  lg   |  xl
      */
-    const keys = [...theme.breakpoints.keys].reverse();
+    const keys = theme.breakpoints.keys.slice().reverse();
     const widthComputed = keys.reduce((output, key) => {
       // eslint-disable-next-line react-hooks/rules-of-hooks
       const matches = useMediaQuery(theme.breakpoints.up(key));

diff --git a/test/utils/consoleErrorMock.test.js b/test/utils/consoleErrorMock.test.js
@@ -12,8 +12,7 @@ describe('consoleErrorMock()', () => {
 
   describe('args', () => {
     it('was removed but throws a descriptive error', () => {
-      expect(
-        () => consoleErrorMock.args()).to.throw(
+      expect(() => consoleErrorMock.args()).to.throw(
         'args() was removed in favor of messages(). Use messages() to match against the actual error message that will be displayed in the console.',
       );
     });