Material-ui: [MenuItem] onClick still fired when disabled

[levrik]

Problem description

The onClick event of a MenuItem is still being fired if disabled is set to true.

Link to minimal working code that reproduces the issue

Webpackbin seems broken atm 😢

Versions

• Material-UI: 0.17.1
• React: 15.4.2
• Browser: Chrome

[levrik]

Opened a issue regarding the Webpackbin issue over there: https://github.com/christianalfoni/webpack-bin/issues/236

[levrik]

Okay. webpackbin is fixed. Here's a bin: https://www.webpackbin.com/bins/-Kh6UKEuSe12HL5xmpuY

[mbrookes]

@levrik onTouchTap: https://www.webpackbin.com/bins/-Kh7-nn6DXivDNS4m5FK

[kpala]

onTouchTap seems to fire on disabled MenuItems with Material-UI 0.18.5: https://www.webpackbin.com/bins/-KoIFGpNmaNZLiMZkslA

[bbosiljcic]

this issue still persist, I don't get why this was removed

[bbosiljcic]

this issue was fixed in 0.18.7

[Nikhil-Shenoy]

I am also encountering this issue, and would like a fix as soon as possible. Using 0.19.4

[mbrookes]

@Nikhil-Shenoy We are focusing on the v1-beta branch. If this issue is still occurring, you're welcome to submit a PR.

[pavanshinde47]

Still see the issue