Fix "destory" method in view plugins.

[KaneCohen]

Update destory methods on BubbleMenu and FloatingMenu plugins to correctly remove event listeners.

[philippkuehn]

what do you mean by correctly remove event listeners? is there any issue that you can explain? thanks!

[KaneCohen]

Mousedown event is added with a "capture" flag:

 this.element.addEventListener('mousedown', this.mousedownHandler, { capture: true })

In order to remove event listener same "signature" of the event should be used, including capture flag. Currently this is not the case:

this.element.removeEventListener('mousedown', this.mousedownHandler)

Which leads to mousedown event not being removed.

From MDN Web Docs:

The EventTarget.removeEventListener() method removes from the EventTarget an event listener previously registered with EventTarget.addEventListener(). The event listener to be removed is identified using a combination of the event type, the event listener function itself, and various optional options that may affect the matching process; see Matching event listeners for removal

[philippkuehn]

ah great! thanks!