[Select] onClick not fired on menu items

• The issue is present in the latest release.
• I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

If you select the same value twice in a single Select, onChange is not called the second time.

Expected Behavior 🤔

onChange should be called every time a user selects a value, even if it’s the same value as before.

Steps to Reproduce 🕹

Here’s an example of functionality that worked in v4.9.8: https://codesandbox.io/s/frosty-cohen-gpk3y?file=/src/Demo.js

Here’s the same code not working in v4.9.9: https://codesandbox.io/s/hungry-neumann-y0ht0?file=/src/Demo.js

Steps:

1. Click on the Select
2. Select the value that is already selected

Context 🔦

We prefer to have the single Select behave in the same way as the multi Select- click on an already selected item to deselect it. Without an event, we can’t do that.

Related issue: https://github.com/mui-org/material-ui/issues/20351 Related PR: https://github.com/mui-org/material-ui/pull/20361

Your Environment 🌎

[oliviertassinari]

<kbd>Enter</kbd> will trigger onClick, we are good.

[tpeek]

I think we might have to do a bit more with the onKeyUp part- I’ve tested passing onKeyUp to each MenuItem, and it works for the arrow keys but not for the ‘Enter’ key. Example: https://codesandbox.io/s/divine-violet-jucyq?file=/src/Demo.js

My guess is that the Menu closes onKeyDown, so the MenuItem isn’t around for the onKeyUp event.