Add `disabled` to `ContextMenu`

[Andarist]

Feature request

It would be nice if ContextMenu could be disabled. We have a situation in which we conditionally want to disable the context menu - we can do that by conditionally rendering the ContextMenu wrapper but that remounts the wrapped tree.

[abbazabacto]

While waiting for disabled to arrive, I found an alternative way which doesn't require conditional rendering of the ContextMenu

It is possible to preventDefault the onContextMenu event on the Trigger. This will effectively not render the ContextMenu

<ContextMenuPrimitive.Trigger onContextMenu={e => disabled && e.preventDefault()}

[Andarist]

This also prevents the native context menu. I think that this is usually not something that you want. I'd be surprised, as a user, if my right click would not invoke anything at all.

[abbazabacto]

You are right, this is a case where I do want to prevent the native context menu.
Understand this is my solution, instead of the coming disabled feature. Thanks for your input!

Edit: For UX we decided to just show some context menu with warning for the "disabled" case. Doing nothing would be weird for the user, as well as showing the browser context menu as the user would "expect" a custom one.