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/02/08 14:44:30 UTC

[royale-asjs] branch develop updated: make Jewel ToolTip only available on all devices except iOS. This fixes the issue of blocking click events on first touch and need the user to do a second touch. This is caused by MouseEvent.MOUSEOVER.

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 bc4a912  make Jewel ToolTip only available on all devices except iOS. This fixes the issue of blocking click events on first touch and need the user to do a second touch. This is caused by MouseEvent.MOUSEOVER.
bc4a912 is described below

commit bc4a912c964e8c213e0ecef9da1bcf4370ef1d96
Author: Carlos Rovira <ca...@apache.org>
AuthorDate: Fri Feb 8 15:44:23 2019 +0100

    make Jewel ToolTip only available on all devices except iOS. This fixes the issue of blocking click events on first touch and need the user to do a second touch. This is caused by MouseEvent.MOUSEOVER.
---
 .../main/royale/org/apache/royale/jewel/beads/controls/ToolTip.as  | 7 ++++++-
 .../org/apache/royale/jewel/debugger/ResponsiveSizeMonitor.as      | 5 +----
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controls/ToolTip.as b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controls/ToolTip.as
index 007bed8..d8b1c6b 100644
--- a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controls/ToolTip.as
+++ b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controls/ToolTip.as
@@ -27,6 +27,7 @@ package org.apache.royale.jewel.beads.controls
 	import org.apache.royale.events.MouseEvent;
 	import org.apache.royale.geom.Point;
 	import org.apache.royale.jewel.supportClasses.tooltip.ToolTipLabel;
+	import org.apache.royale.utils.OSUtils;
 	import org.apache.royale.utils.PointUtils;
 	import org.apache.royale.utils.UIUtils;
 
@@ -127,7 +128,11 @@ package org.apache.royale.jewel.beads.controls
 		{
 			_strand = value;
 
-            IEventDispatcher(_strand).addEventListener(MouseEvent.MOUSE_OVER, rollOverHandler, false);
+			//ToolTip in iOS produces a bad behaviour, used in a button and user has to do a second touch to trigger click event
+			if(OSUtils.getOS() != OSUtils.IOS_OS)
+			{
+            	IEventDispatcher(_strand).addEventListener(MouseEvent.MOUSE_OVER, rollOverHandler, false);
+			}
 		}
 
 		/**
diff --git a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/debugger/ResponsiveSizeMonitor.as b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/debugger/ResponsiveSizeMonitor.as
index abcbf6d..64a7e58 100644
--- a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/debugger/ResponsiveSizeMonitor.as
+++ b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/debugger/ResponsiveSizeMonitor.as
@@ -18,12 +18,9 @@
 ////////////////////////////////////////////////////////////////////////////////
 package org.apache.royale.jewel.debugger
 {	
-	import org.apache.royale.core.IBead;
-	import org.apache.royale.core.IStrand;
 	import org.apache.royale.events.Event;
-	import org.apache.royale.jewel.Drawer;
-	import org.apache.royale.jewel.supportClasses.ResponsiveSizes;
 	import org.apache.royale.jewel.Label;
+	import org.apache.royale.jewel.supportClasses.ResponsiveSizes;
 	
 	/**
 	 *  The ResponsiveDrawer class is a bead to use with Jewel Drawer to make it fixed or