[Select][material-ui] Respect default prevented event on click (do not close)

[pcorpet]

Summary

When using event.preventDefault() inside a MenuItem's onClick callback, it should not change the value nor close the menu.

Examples

Clicking the "load more" option below could keep the Select menu open.

<Select value={age} label="Age" onChange={handleChange}>
    <MenuItem value={10}>Ten</MenuItem>
    <MenuItem value={20}>Twenty</MenuItem>
    <MenuItem value={30}>Thirty</MenuItem>
    <MenuItem
        onClick={e => {
            e.preventDefault()
            loadMoreAgeOptions()
        }>load more ...</MenuItem>
</Select>

There is a workaround where I can keep the open state controlled and decide to ignore the close, however this seems a good use of the preventDefault feature while making it way more streamlined to implement.

Motivation

No response

Search keywords: Select preventDefault

[DiegoAndai]

Hey @pcorpet, thanks for reaching out. Would you consider this to be a sub-case / duplicate of #26356? If #26356 was implemented, you could add the "Load more" children that wouldn't be an actual option. Does this make sense?

[pcorpet]

Yes, that'd work for me.

[DiegoAndai]

Thanks for the reply. I will close this one in favor of #26356 to keep upvotes concentrated in one place. Please upvote #26356 so we can prioritize it eventually.