[royale-asjs] branch develop updated: jewel-button: add compile::js around royale event to avoid IDEs remove it when organize

[ca...@apache.org]

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 c145595  jewel-button: add compile::js around royale event to avoid IDEs remove it when organize
c145595 is described below

commit c14559588475e038a77b0ca37f35c2109bbff09c
Author: Carlos Rovira <ca...@apache.org>
AuthorDate: Sat Dec 14 19:23:35 2019 +0100

    jewel-button: add compile::js around royale event to avoid IDEs remove it when organize
---
 .../Jewel/src/main/royale/org/apache/royale/jewel/Button.as         | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/Button.as b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/Button.as
index 4a8fb27..8c5693e 100644
--- a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/Button.as
+++ b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/Button.as
@@ -22,9 +22,13 @@ package org.apache.royale.jewel
     {
     import org.apache.royale.core.ITextModel;
     }
-    //Below is needed to avoid dispatching native JS Event, which seems to still work in Chrome and probably other browsers,
+    COMPILE::JS
+    {
+    //@gregdove: Below is needed to avoid dispatching native JS Event, which seems to still work in Chrome and probably other browsers,
     //but IE11 will break if the following import is removed:
+    //@carlosrovira: Probably the above issue is due to the need to be in COMPILE::JS, so IDEs organizing imports don't remove this one
     import org.apache.royale.events.Event;
+    }
     import org.apache.royale.jewel.supportClasses.button.SimpleButton;
 
     [DefaultProperty("text")]