SpeedDialAction requires undocumented onClick handler on touch screens

[haydengomes]

• This is not a v0.x issue.
• I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

Configuring the SpeedDialAction button to use a Link component should update route on click on both desktop and touch screen devices.

Current Behavior

On touch screens, an exception is thrown because the SpeedDialAction is expecting an onClick function prop to be defined.

Steps to Reproduce

https://codesandbox.io/s/q9q7jrjr6j

1. Create a SpeedDialAction with ButtonProps that configure the component to be Link and the to to be some valid route.
2. In desktop, click on the action button and see that you're taken to the expected route
3. On a touch enabled device, or Chrome's mobile emulator, tap on the same action button

Link:

1. https://github.com/mui-org/material-ui/blob/master/packages/material-ui-lab/src/SpeedDialAction/SpeedDialAction.js#L91
2. https://github.com/mui-org/material-ui/blob/master/pages/lab/api/speed-dial-action.md
3. https://codesandbox.io/s/q9q7jrjr6j

Context

I created a SpeedDial and used ButtonProps to configure the SpeedDialAction buttons to use the Link component and have a to destination route for when it gets clicked. This worked fine on desktop, but on mobile an exception is thrown because the onClick function is not defined at
SpeedDialAction.js

(This prop seems to be missing from the documentation SpeedDialAction.md)

Your Environment

Tech	Version
Material-UI	^3.2.2
Material-UI lab	^3.0.0-alpha.23
React	^16.5.2
Chrome	70.0.3538.77
react-router-dom	^4.3.1

[balinabbb]

I was about to submit the very same issue, with one addition:

I get the following warning after navigating away with component={Link} and to="/other-url".
Did not test it, but my guess is the setState in handleTooltipClose after navigating away and unmounting the component.

React warning

Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in the componentWillUnmount method.

    in SpeedDialAction (created by ForwardRef(SpeedDialAction))
    in ForwardRef(SpeedDialAction) (created by WithStyles(ForwardRef(SpeedDialAction)))
    in WithStyles(ForwardRef(SpeedDialAction)) (at Component.jsx:35)
    in div (created by ForwardRef(SpeedDial))
    in div (created by ForwardRef(SpeedDial))
    in ForwardRef(SpeedDial) (created by WithStyles(ForwardRef(SpeedDial)))

Your Environment 🌎

Tech	Version
Material-UI	v4.0.0
React	v16.8.6
Browser	Chromium 75.0.3770.90 (Official Build) snap (64-bit)
Material-UI lab	v4.0.0-alpha.18