From b95b4d2b2f6f76752d74a29c2fb2aa173be59830 Mon Sep 17 00:00:00 2001 From: Andrea Bogazzi Date: Mon, 25 Apr 2016 10:32:23 +0200 Subject: [PATCH] Search the target again on mouse up --- src/mixins/canvas_events.mixin.js | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/mixins/canvas_events.mixin.js b/src/mixins/canvas_events.mixin.js index 4ce433da..14755d74 100644 --- a/src/mixins/canvas_events.mixin.js +++ b/src/mixins/canvas_events.mixin.js @@ -249,21 +249,20 @@ * @param {Event} e Event object fired on mouseup */ __onMouseUp: function (e) { - var target; + var target, searchTarget = true, transform = this._currentTransform; if (this.isDrawingMode && this._isCurrentlyDrawing) { this._onMouseUpInDrawingMode(e); return; } - if (this._currentTransform) { + if (transform) { this._finalizeCurrentTransform(); - target = this._currentTransform.target; - } - else { - target = this.findTarget(e, true); + searchTarget = !transform.actionPerformed; } + target = searchTarget ? this.findTarget(e, true) : transform.target; + var shouldRender = this._shouldRender(target, this.getPointer(e)); this._maybeGroupObjects(e); @@ -280,11 +279,12 @@ _handleCursorAndEvent: function(e, target) { this._setCursorFromEvent(e, target); + // Can't find any reason, disabling for now // TODO: why are we doing this? - var _this = this; + /* var _this = this; setTimeout(function () { _this._setCursorFromEvent(e, target); - }, 50); + }, 50); */ this.fire('mouse:up', { target: target, e: e }); target && target.fire('mouseup', { e: e });