[core] Remove some IE11 support

mui · Mar 23, 2024 · 2c8da32 · 2c8da32
1 parent 0102a95
commit 2c8da32
Show file tree

Hide file tree

Showing 9 changed files with 12 additions and 59 deletions.
diff --git a/packages/mui-base/src/useAutocomplete/useAutocomplete.js b/packages/mui-base/src/useAutocomplete/useAutocomplete.js
@@ -10,11 +10,8 @@ import {
 } from '@mui/utils';
 
 // https://stackoverflow.com/questions/990904/remove-accents-diacritics-in-a-string-in-javascript
-// Give up on IE11 support for this feature
 function stripDiacritics(string) {
-  return typeof string.normalize !== 'undefined'
-    ? string.normalize('NFD').replace(/[\u0300-\u036f]/g, '')
-    : string;
+  return string.normalize('NFD').replace(/[\u0300-\u036f]/g, '');
 }
 
 export function createFilterOptions(config = {}) {
@@ -56,17 +53,6 @@ export function createFilterOptions(config = {}) {
   };
 }
 
-// To replace with .findIndex() once we stop IE11 support.
-function findIndex(array, comp) {
-  for (let i = 0; i < array.length; i += 1) {
-    if (comp(array[i])) {
-      return i;
-    }
-  }
-
-  return -1;
-}
-
 const defaultFilterOptions = createFilterOptions();
 
 // Number of options to jump in list box when `Page Up` and `Page Down` keys are used.
@@ -498,7 +484,7 @@ export function useAutocomplete(props) {
       const previousHighlightedOption = previousProps.filteredOptions[highlightedIndexRef.current];
 
       if (previousHighlightedOption) {
-        return findIndex(filteredOptions, (option) => {
+        return filteredOptions.findIndex((option) => {
           return getOptionLabel(option) === getOptionLabel(previousHighlightedOption);
         });
       }
@@ -539,12 +525,12 @@ export function useAutocomplete(props) {
       if (
         multiple &&
         currentOption &&
-        findIndex(value, (val) => isOptionEqualToValue(currentOption, val)) !== -1
+        value.findIndex((val) => isOptionEqualToValue(currentOption, val)) !== -1
       ) {
         return;
       }
 
-      const itemIndex = findIndex(filteredOptions, (optionItem) =>
+      const itemIndex = filteredOptions.findIndex((optionItem) =>
         isOptionEqualToValue(optionItem, valueItem),
       );
       if (itemIndex === -1) {
@@ -685,7 +671,7 @@ export function useAutocomplete(props) {
         }
       }
 
-      const itemIndex = findIndex(newValue, (valueItem) => isOptionEqualToValue(option, valueItem));
+      const itemIndex = newValue.findIndex((valueItem) => isOptionEqualToValue(option, valueItem));
 
       if (itemIndex === -1) {
         newValue.push(option);

diff --git a/packages/mui-base/src/useSnackbar/useSnackbar.ts b/packages/mui-base/src/useSnackbar/useSnackbar.ts
@@ -44,8 +44,7 @@ export function useSnackbar(parameters: UseSnackbarParameters = {}): UseSnackbar
      */
     function handleKeyDown(nativeEvent: KeyboardEvent) {
       if (!nativeEvent.defaultPrevented) {
-        // IE11, Edge (prior to using Blink?) use 'Esc'
-        if (nativeEvent.key === 'Escape' || nativeEvent.key === 'Esc') {
+        if (nativeEvent.key === 'Escape') {
           // not calling `preventDefault` since we don't know if people may ignore this event e.g. a permanently open snackbar
           onClose?.(nativeEvent, 'escapeKeyDown');
         }

diff --git a/packages/mui-joy/src/Tooltip/Tooltip.tsx b/packages/mui-joy/src/Tooltip/Tooltip.tsx
@@ -409,8 +409,7 @@ const Tooltip = React.forwardRef(function Tooltip(inProps, ref) {
     }
 
     function handleKeyDown(nativeEvent: KeyboardEvent) {
-      // IE11, Edge (prior to using Bink?) use 'Esc'
-      if (nativeEvent.key === 'Escape' || nativeEvent.key === 'Esc') {
+      if (nativeEvent.key === 'Escape') {
         handleClose(nativeEvent);
       }
     }

diff --git a/packages/mui-material/src/Tooltip/Tooltip.js b/packages/mui-material/src/Tooltip/Tooltip.js
@@ -473,8 +473,7 @@ const Tooltip = React.forwardRef(function Tooltip(inProps, ref) {
      * @param {KeyboardEvent} nativeEvent
      */
     function handleKeyDown(nativeEvent) {
-      // IE11, Edge (prior to using Bink?) use 'Esc'
-      if (nativeEvent.key === 'Escape' || nativeEvent.key === 'Esc') {
+      if (nativeEvent.key === 'Escape') {
         handleClose(nativeEvent);
       }
     }

diff --git a/packages/mui-material/src/styles/createTransitions.js b/packages/mui-material/src/styles/createTransitions.js
@@ -64,9 +64,7 @@ export default function createTransitions(inputTransitions) {
 
     if (process.env.NODE_ENV !== 'production') {
       const isString = (value) => typeof value === 'string';
-      // IE11 support, replace with Number.isNaN
-      // eslint-disable-next-line no-restricted-globals
-      const isNumber = (value) => !isNaN(parseFloat(value));
+      const isNumber = (value) => !Number.isNaN(parseFloat(value));
       if (!isString(props) && !Array.isArray(props)) {
         console.error('MUI: Argument "props" must be a string or Array.');
       }

diff --git a/packages/mui-utils/src/getDisplayName/getDisplayName.test.tsx b/packages/mui-utils/src/getDisplayName/getDisplayName.test.tsx
@@ -1,7 +1,7 @@
 /* eslint-disable react/prefer-stateless-function */
 import * as React from 'react';
 import { expect } from 'chai';
-import getDisplayName, { getFunctionName } from './getDisplayName';
+import getDisplayName from './getDisplayName';
 
 describe('utils/getDisplayName.js', () => {
   describe('getDisplayName', () => {
@@ -75,14 +75,4 @@ describe('utils/getDisplayName.js', () => {
       expect(getDisplayName(false)).to.equal(undefined);
     });
   });
-
-  describe('getFunctionName', () => {
-    it('gets the name of a function', () => {
-      function SomeFunction() {
-        return <div />;
-      }
-
-      expect(getFunctionName(SomeFunction)).to.equal('SomeFunction');
-    });
-  });
 });
diff --git a/packages/mui-utils/src/getDisplayName/getDisplayName.ts b/packages/mui-utils/src/getDisplayName/getDisplayName.ts
@@ -1,20 +1,11 @@
 import * as React from 'react';
 import { ForwardRef, Memo } from 'react-is';
 
-// Simplified polyfill for IE11 support
-// https://github.com/JamesMGreene/Function.name/blob/58b314d4a983110c3682f1228f845d39ccca1817/Function.name.js#L3
-const fnNameMatchRegex = /^\s*function(?:\s|\s*\/\*.*\*\/\s*)+([^(\s/]*)\s*/;
-export function getFunctionName(fn: Function): string {
-  const match = `${fn}`.match(fnNameMatchRegex);
-  const name = match && match[1];
-  return name || '';
-}
-
 function getFunctionComponentName(
   Component: React.FunctionComponent | React.ComponentClass,
   fallback = '',
 ) {
-  return Component.displayName || Component.name || getFunctionName(Component) || fallback;
+  return Component.displayName || Component.name || fallback;
 }
 
 function getWrappedName(outerType: any, innerType: any, wrapperName: string) {

diff --git a/packages/mui-utils/src/getDisplayName/index.ts b/packages/mui-utils/src/getDisplayName/index.ts
@@ -1,2 +1 @@
 export { default } from './getDisplayName';
-export * from './getDisplayName';
diff --git a/packages/mui-utils/src/integerPropType/integerPropType.js b/packages/mui-utils/src/integerPropType/integerPropType.js
@@ -24,18 +24,10 @@ export function getTypeByValue(value) {
   }
 }
 
-// IE 11 support
-function ponyfillIsInteger(x) {
-  // eslint-disable-next-line no-restricted-globals
-  return typeof x === 'number' && isFinite(x) && Math.floor(x) === x;
-}
-
-const isInteger = Number.isInteger || ponyfillIsInteger;
-
 function requiredInteger(props, propName, componentName, location) {
   const propValue = props[propName];
 
-  if (propValue == null || !isInteger(propValue)) {
+  if (propValue == null || !Number.isInteger(propValue)) {
     const propType = getTypeByValue(propValue);
 
     return new RangeError(
Original file line number	Diff line number	Diff line change
		@@ -1,2 +1 @@
		export { default } from './getDisplayName';
		export * from './getDisplayName';