fix(mvc.Dom): remove event listeners on element removal; fix(linkTool…

…s.Vertices): fix to fire mouseleave event when redundancyRemoval is disabled (#2482)

packages/joint-core/src/mvc/Dom/methods.mjs

@@ -1,35 +1,50 @@
 import { camelCase } from '../../util/utilHelpers.mjs';
 import $ from './Dom.mjs';
 import V from '../../V/index.mjs';
-import { dataPriv, cleanNodesData } from './vars.mjs';
+import { dataPriv, dataUser } from './vars.mjs';
 
 // Manipulation
 
-function removeNodes(nodes, removeData) {
+function cleanNodesData(nodes) {
+    let i = nodes.length;
+    while (i--) cleanNodeData(nodes[i]);
+}
+
+function cleanNodeData(node) {
+    $.event.remove(node);
+    dataPriv.remove(node);
+    dataUser.remove(node);
+}
+
+function removeNodes(nodes) {
     for (let i = 0; i < nodes.length; i++) {
         const node = nodes[i];
-        removeData && dataPriv.remove(node);
         if (node.parentNode) {
             node.parentNode.removeChild(node);
         }
     }
 }
 
 export function remove() {
-    removeNodes(this, true);
+    for (let i = 0; i < this.length; i++) {
+        const node = this[i];
+        cleanNodeData(node);
+        cleanNodesData(node.getElementsByTagName('*'));
+    }
+    removeNodes(this);
     return this;
 }
 
 export function detach() {
-    removeNodes(this, false);
+    removeNodes(this);
     return this;
 }
 
 export function empty() {
     for (let i = 0; i < this.length; i++) {
         const node = this[i];
         if (node.nodeType === 1) {
-            cleanNodesData(dataPriv, node.getElementsByTagName('*'));
+            cleanNodesData(node.getElementsByTagName('*'));
             // Remove any remaining nodes
             node.textContent = '';
         }

packages/joint-core/src/mvc/Dom/vars.mjs

@@ -3,10 +3,3 @@ import Data from '../Data.mjs';
 export const dataPriv = new Data();
 
 export const dataUser = new Data();
-
-export function cleanNodesData(data, nodes) {
-    let i = nodes.length;
-    while (i--) {
-        data.remove(nodes[i]);
-    }
-}