Remove unnecessary useEventCallback

mui · Jun 15, 2021 · a60685e · a60685e
1 parent c80a4b1
commit a60685e
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/packages/material-ui/src/ButtonBase/Ripple.js b/packages/material-ui/src/ButtonBase/Ripple.js
@@ -1,7 +1,6 @@
 import * as React from 'react';
 import PropTypes from 'prop-types';
 import clsx from 'clsx';
-import useEventCallback from '../utils/useEventCallback';
 
 /**
  * @ignore - internal component.
@@ -15,7 +14,7 @@ function Ripple(props) {
     rippleY,
     rippleSize,
     in: inProp,
-    onExited = () => {},
+    onExited,
     timeout,
   } = props;
   const [leaving, setLeaving] = React.useState(false);
@@ -36,20 +35,19 @@ function Ripple(props) {
     [classes.childPulsate]: pulsate,
   });
 
-  const handleExited = useEventCallback(onExited);
   if (!inProp && !leaving) {
     setLeaving(true);
   }
   React.useEffect(() => {
-    if (!inProp) {
+    if (!inProp && onExited != null) {
       // react-transition-group#onExited
-      const timeoutId = setTimeout(handleExited, timeout);
+      const timeoutId = setTimeout(onExited, timeout);
       return () => {
         clearTimeout(timeoutId);
       };
     }
     return undefined;
-  }, [handleExited, inProp, timeout]);
+  }, [onExited, inProp, timeout]);
 
   return (
     <span className={rippleClassName} style={rippleStyles}>