[TouchRipple] Allows call imperative methods without event

[alexfauquette]

• I have followed (at least) the PR section of the contributing guide.

According to the typescript interface of the TouchRipple the ref should allows to call start and stop without the event argument.

export interface TouchRippleActions {
  start: (
    event?: React.SyntheticEvent,
    options?: StartActionOptions,
    callback?: () => void,
  ) => void;
  pulsate: (event?: React.SyntheticEvent) => void;
  stop: (event?: React.SyntheticEvent, callback?: () => void) => void;
}

But events properties such as event.type are used in start and stop methods which makes them return an error. WHen calling rippleRef.current.stop()

[mui-bot]

Details of bundle changes

Generated by 🚫 dangerJS against 6318b5e

[mnajdova]

@alexfauquette can you maybe share a sandbox where this fails? As far as I can see in the method that is updated the event has a default value, which is an {}.

[alexfauquette]

@mnajdova Effectively, my report is unclear

Here is a minimal reproduction. With rippleRef.current.stop({}) it works, but with rippleRef.current.stop() I receive an error

https://codesandbox.io/s/basicbuttons-material-demo-forked-l1rd3u?file=/demo.tsx

[mnajdova]

Thanks for sharing the sandbox, it explains things :)