Fix #9863: ContextMenu cleanup DOM events on destroy

[melloware]

Fix #9863: ContextMenu cleanup DOM events on destroy

This adds the ability to remove all the subscribed DOM events from the document on destroy but in the scenario in the ticket I don't see why destroy is not being called? cc @tandraschko maybe i just don't understand enough about the destroy method.

[tandraschko]

destroy is called when the widget does not exist anymore after the ajax request
in case it still exists, refresh is called
didnt check your case above

[melloware]

Yep it gets rendered="false" so it should be removed from the DOM "I think".

[tandraschko]

thats weird then
can you check its called for older widgets?
hope nothing is broken in jquery ui core

[melloware]

Yep I will do some research I just wanted to make sure I understood how its supposed to work. I will let you know what I find out.

[melloware]

OK the issue is ContextMenu is attached to the document.body so it never calls destroy because

if (widget && widget.isDetached() === true) {
                        widget.destroy();

The isDetached will never be true as the Context Menu is always in the DOM.