From 54c59238f5fdbf2fef1df73ce33fcfbb01fec23c Mon Sep 17 00:00:00 2001
From: jake downs <mail@jakedowns.com>
Date: Tue, 3 Dec 2019 09:47:18 -0800
Subject: [PATCH] Fixes #4115 - triggers in/out events for sub targets; and
 calls _setCursorFromEvent for sub targets

pass linting
---
 src/mixins/canvas_events.mixin.js | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/src/mixins/canvas_events.mixin.js b/src/mixins/canvas_events.mixin.js
index c88ad098deb..6dd093dd74b 100644
--- a/src/mixins/canvas_events.mixin.js
+++ b/src/mixins/canvas_events.mixin.js
@@ -777,7 +777,7 @@
     __onMouseMove: function (e) {
       this._handleEvent(e, 'move:before');
       this._cacheTransformEventData(e);
-      var target, pointer;
+      var target, pointer, _this = this;
 
       if (this.isDrawingMode) {
         this._onMouseMoveInDrawingMode(e);
@@ -803,6 +803,14 @@
         target = this.findTarget(e) || null;
         this._setCursorFromEvent(e, target);
         this._fireOverOutEvents(target, e);
+        // handle triggering on SubTargets
+        this.targets.map(function(subTarget,k){
+          _this._fireOverOutEvents(subTarget, e, '_hoveredTarget' + (_this.targets.length - k - 1));
+        });
+        // hoverCursor should come from top-most subtarget
+        this.targets.slice(0).reverse().map(function(subTarget){
+          _this._setCursorFromEvent(e, subTarget);
+        });
       }
       else {
         this._transformObject(e);
@@ -815,11 +823,12 @@
      * Manage the mouseout, mouseover events for the fabric object on the canvas
      * @param {Fabric.Object} target the target where the target from the mousemove event
      * @param {Event} e Event object fired on mousemove
+     * @param {String} targetName property on the canvas where the target is stored
      * @private
      */
-    _fireOverOutEvents: function(target, e) {
+    _fireOverOutEvents: function(target, e, targetName) {
       this.fireSyntheticInOutEvents(target, e, {
-        targetName: '_hoveredTarget',
+        targetName: targetName || '_hoveredTarget',
         canvasEvtOut: 'mouse:out',
         evtOut: 'mouseout',
         canvasEvtIn: 'mouse:over',
@@ -866,6 +875,7 @@
       if (outFires) {
         canvasEvtOut && this.fire(canvasEvtOut, outOpt);
         oldTarget.fire(config.evtOut, outOpt);
+        delete this[config.targetName]; // de-reference old target to prevent leaks
       }
       if (inFires) {
         canvasEvtIn && this.fire(canvasEvtIn, inOpt);