You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@royale.apache.org by ca...@apache.org on 2019/04/17 21:53:29 UTC

[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

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 <ca...@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);
 			}
 		}