[SpeedDial] Actions container takes has full height when closed (and trigger hover event)

[araujoigor]

• This is a v1.x issue (v0.x is no longer maintained).
• I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

When the SpeedDial is closed (no actions being shown) the action container should be fully colapsed and have no height, not triggering events such as hover, for example (only the FAB itself should trigger the hover).

Current Behavior

When the SpeedDial is closed (no actions being shown) the action container still has the same height as when it is open and therefore trigger onHover on places it shouldn't (eventually preventing actions on elements "behind" it.

Steps to Reproduce (for bugs)

https://codesandbox.io/s/k00ymzjj5v

1. Hover a few pixels above the FAB
2. Observe that the actions show up since the "empty" container still had its full-height

Context

I'm using the component for a personal project and believe that triggering the hover effect on the "invisible" action elements is a bad UX and a unwanted behavior.

Your Environment

Tech	Version
Material-UI	^1.2.1
React	^16.4.0
browser	All, browser independent issue - Safari 11.1, Chrome 67, Firefox 61

[mbrookes]

triggering the hover effect on the "invisible" action elements is a bad UX

@araujoigor I had the same thought, but I left it because of: https://inbox.google.com/ (Not saying they are right of course!)

Do you want to work on it?

[araujoigor]

@mbrookes Interesting... Never realized Inbox had such behavior... I assume it is because they leave the entire "row" for the FAB and its actions... But still, I don't like much.

Sure, I can work on that. Any suggestions on the implementation? Since transitioning display is not an option, the only two solutions that come to my mind are either to transition max-height (which is a bit hacky and could possibly lead to undesired effects and because we limit the height) or add a callback with setTimeout to add a class with display: none.

[mbrookes]

We could possibly unmount the actions when they're closed, although not sure what that would do for performance when you hover over the FAB. (Should be okay, given they animate in).

The only alternative I can think of would be to proxy mouse events from the FAB...