Move helper function to end + remove unnecessary variable

mui · Nov 30, 2024 · cf54828 · cf54828
1 parent 762daf7
commit cf54828
Showing 1 changed file with 14 additions and 15 deletions.
diff --git a/packages/mui-material/src/ButtonBase/ButtonBase.js b/packages/mui-material/src/ButtonBase/ButtonBase.js
@@ -68,21 +68,6 @@ export const ButtonBaseRoot = styled('button', {
   },
 });
 
-function useRippleHandler(ripple, rippleAction, eventCallback, skipRippleAction = false) {
-  return useEventCallback((event) => {
-    if (eventCallback) {
-      eventCallback(event);
-    }
-
-    const ignore = skipRippleAction;
-    if (!ignore) {
-      ripple[rippleAction](event);
-    }
-
-    return true;
-  });
-}
-
 /**
  * `ButtonBase` contains as few styles as possible.
  * It aims to be a simple building block for creating a button.
@@ -332,6 +317,20 @@ const ButtonBase = React.forwardRef(function ButtonBase(inProps, ref) {
   );
 });
 
+function useRippleHandler(ripple, rippleAction, eventCallback, skipRippleAction = false) {
+  return useEventCallback((event) => {
+    if (eventCallback) {
+      eventCallback(event);
+    }
+
+    if (!skipRippleAction) {
+      ripple[rippleAction](event);
+    }
+
+    return true;
+  });
+}
+
 ButtonBase.propTypes /* remove-proptypes */ = {
   // ┌────────────────────────────── Warning ──────────────────────────────┐
   // │ These PropTypes are generated from the TypeScript type definitions. │