fix(linkTools): fix pending batch for TargetArrowhead and SourceArrow…

…head
clientIO · Jul 24, 2024 · bd2f631 · bd2f631
1 parent b1b7a45
commit bd2f631
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 1 deletion.
diff --git a/packages/joint-core/src/linkTools/Arrowhead.mjs b/packages/joint-core/src/linkTools/Arrowhead.mjs
@@ -51,7 +51,8 @@ const Arrowhead = ToolView.extend({
         var paper = relatedView.paper;
         relatedView.model.startBatch('arrowhead-move', { ui: true, tool: this.cid });
         relatedView.startArrowheadMove(this.arrowheadType);
-        this.delegateDocumentEvents();
+        const data = evt.data || (evt.data = {});
+        this.delegateDocumentEvents(null, data);
         paper.undelegateEvents();
         this.focus();
         this.el.style.pointerEvents = 'none';

diff --git a/packages/joint-core/test/jointjs/dia/linkTools.js b/packages/joint-core/test/jointjs/dia/linkTools.js
@@ -341,6 +341,30 @@ QUnit.module('linkTools', function(hooks) {
 
             listener.stopListening();
         });
+
+        QUnit.test('data', function(assert) {
+
+            assert.expect(3);
+
+            const arrowhead = new joint.linkTools.TargetArrowhead();
+            linkView.addTools(new joint.dia.ToolsView({ tools: [arrowhead] }));
+            const listener = new joint.mvc.Listener();
+
+            listener.listenTo(paper, {
+                'link:pointerdown': (linkView, evt) => evt.data.test = true,
+                'link:pointermove': (linkView, evt) => assert.ok(evt.data.test),
+                'link:pointerup': (linkView, evt) => assert.ok(evt.data.test),
+            });
+
+            // Make sure the link is connected to a target element
+            assert.ok(link.getTargetCell());
+
+            simulate.mousedown({ el: arrowhead.el });
+            simulate.mousemove({ el: paper.el });
+            simulate.mouseup({ el: paper.el });
+
+            listener.stopListening();
+        });
     });
 
 });