[TouchRipple] perf: avoid calling clearTimeout() (#37512)

mui · Jun 21, 2023 · 6167b6a · 6167b6a
1 parent a91427e
commit 6167b6a
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/packages/mui-material/src/ButtonBase/TouchRipple.js b/packages/mui-material/src/ButtonBase/TouchRipple.js
@@ -136,15 +136,17 @@ const TouchRipple = React.forwardRef(function TouchRipple(inProps, ref) {
   const ignoringMouseDown = React.useRef(false);
   // We use a timer in order to only show the ripples for touch "click" like events.
   // We don't want to display the ripple for touch scroll events.
-  const startTimer = React.useRef(null);
+  const startTimer = React.useRef(0);
 
   // This is the hook called once the previous timeout is ready.
   const startTimerCommit = React.useRef(null);
   const container = React.useRef(null);
 
   React.useEffect(() => {
     return () => {
-      clearTimeout(startTimer.current);
+      if (startTimer.current) {
+        clearTimeout(startTimer.current);
+      }
     };
   }, []);