adding support for mouse events to IconMenu

[chrisgervang]

I believe it was intended for this to work with mouse events as well as
touch events. In my project the mouse events were not working correctly
until I added this bit of code. I tested it when nested in a AppBar’s
iconElementRight and when on it’s own.

[oliviertassinari]

The onTouchTap is also supposed to be triggered by a mouse event.

In my project the mouse events were not working correctly

Can you be more specific?

[chrisgervang]

Sure - this is my usage:

https://github.com/chrisgervang/pickoleon/blob/master/app/src/components/PickleBrowser.jsx#L34-L37

Hmm. What I am curious about is that it seems to work in the hosted docs, but not on my machine. I don't think I'm using it differently than the docs though.

If it means anything, I'm on OSX using latest Chrome release.

[oliviertassinari]

Are you starting the React-tap-event-plugin?
http://material-ui.com/#/get-started/installation

[chrisgervang]

I am not. Seems weird to me that I'd need that if I'm not using touch in my app.

Upon first read of the install instructions I thought that was an optional thing, but looking at it again it does seem more like a requirement for now.

[oliviertassinari]

Yes, that's required. The main benefit of it is to remove the 300ms delay of Safari iOS.

I agree, would be better to not be required. Another solution would be to use onClick and to emulate the onClick on Safari iOS

[chrisgervang]

Hmm is that trivial to do?

After following the instructions IconMenu does appear to actually work. I'm going to close this PR because it doesn't seem necessary to fix right now.

[oliviertassinari]

It's not trivial but we could suggest to use https://github.com/ftlabs/fastclick instead.