[royale-asjs] branch develop updated: jewel-itemrenderer: fix listener to use positioner instead of element so we can add more content to the renderers but still handle mouse events at any point inside the render

[carlosrovira]

This is an automated email from the ASF dual-hosted git repository.

carlosrovira pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/royale-asjs.git


The following commit(s) were added to refs/heads/develop by this push:
     new 4a0dd05  jewel-itemrenderer: fix listener to use positioner instead of 
element so we can add more content to the renderers but still handle mouse 
events at any point inside the render
4a0dd05 is described below

commit 4a0dd05f8617183dfb6b1ac88a50fc1a2d05325b
Author: Carlos Rovira <carlosrov...@apache.org>
AuthorDate: Wed Apr 17 23:53:21 2019 +0200

    jewel-itemrenderer: fix listener to use positioner instead of element so we 
can add more content to the renderers but still handle mouse events at any 
point inside the render
---
 .../beads/controllers/ItemRendererMouseController.as  | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controllers/ItemRendererMouseController.as
 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controllers/ItemRendererMouseController.as
index e3447ba..605f4e3 100644
--- 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controllers/ItemRendererMouseController.as
+++ 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controllers/ItemRendererMouseController.as
@@ -18,11 +18,11 @@
 
////////////////////////////////////////////////////////////////////////////////
 package org.apache.royale.jewel.beads.controllers
 {
-COMPILE::SWF {
+       COMPILE::SWF {
        import org.apache.royale.events.Event;
        import org.apache.royale.events.MouseEvent;
-}
-COMPILE::JS {
+       }
+       COMPILE::JS {
        import goog.events;
        import goog.events.Event;
        import goog.events.EventType;
@@ -31,8 +31,7 @@ COMPILE::JS {
        import org.apache.royale.core.UIBase;
        import org.apache.royale.core.WrappedHTMLElement;
        import org.apache.royale.events.BrowserEvent;
-       
-}
+       }
        import org.apache.royale.core.IBeadController;
        import org.apache.royale.core.ISelectableItemRenderer;
        import org.apache.royale.core.IStrand;
@@ -89,12 +88,12 @@ COMPILE::JS {
                        }
                                
                        COMPILE::JS {
-                               var element:WrappedHTMLElement = (_strand as 
UIBase).element;
+                               var positioner:WrappedHTMLElement = (_strand as 
UIBase).positioner;
                                
-                               goog.events.listen(element, 
goog.events.EventType.MOUSEOVER, this.handleMouseOver);
-                               goog.events.listen(element, 
goog.events.EventType.MOUSEOUT, this.handleMouseOut);
-                               goog.events.listen(element, 
goog.events.EventType.MOUSEDOWN, this.handleMouseDown);
-                               goog.events.listen(element, 
goog.events.EventType.CLICK, this.handleMouseUp);
+                               goog.events.listen(positioner, 
goog.events.EventType.MOUSEOVER, this.handleMouseOver);
+                               goog.events.listen(positioner, 
goog.events.EventType.MOUSEOUT, this.handleMouseOut);
+                               goog.events.listen(positioner, 
goog.events.EventType.MOUSEDOWN, this.handleMouseDown);
+                               goog.events.listen(positioner, 
goog.events.EventType.CLICK, this.handleMouseUp);
                        }
                }