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 91cbd1f  move "selectable" and "hoverable" from UIItemRendererBase to 
StyledMXMLItemRenderer for PAYG reasons
91cbd1f is described below

commit 91cbd1f1ff7804dcf90700d8e2b4e17f5a1d4249
Author: Carlos Rovira <[email protected]>
AuthorDate: Fri Feb 15 19:57:47 2019 +0100

    move "selectable" and "hoverable" from UIItemRendererBase to 
StyledMXMLItemRenderer for PAYG reasons
---
 .../apache/royale/core/StyledMXMLItemRenderer.as   | 54 +++++++++++++++++-
 .../html/supportClasses/UIItemRendererBase.as      | 46 ---------------
 .../projects/Core/src/main/royale/CoreClasses.as   |  1 +
 ...nderer.as => IRuntimeSelectableItemRenderer.as} | 65 +---------------------
 .../apache/royale/core/ISelectableItemRenderer.as  | 30 ----------
 .../controllers/ItemRendererMouseController.as     | 17 +++---
 6 files changed, 65 insertions(+), 148 deletions(-)

diff --git 
a/frameworks/projects/Basic/src/main/royale/org/apache/royale/core/StyledMXMLItemRenderer.as
 
b/frameworks/projects/Basic/src/main/royale/org/apache/royale/core/StyledMXMLItemRenderer.as
index baa300d..fa86c50 100644
--- 
a/frameworks/projects/Basic/src/main/royale/org/apache/royale/core/StyledMXMLItemRenderer.as
+++ 
b/frameworks/projects/Basic/src/main/royale/org/apache/royale/core/StyledMXMLItemRenderer.as
@@ -18,9 +18,9 @@
 
////////////////////////////////////////////////////////////////////////////////
 package org.apache.royale.core
 {
+    import org.apache.royale.html.supportClasses.MXMLItemRenderer;
     import org.apache.royale.utils.ClassSelectorList;
     import org.apache.royale.utils.IClassSelectorListSupport;
-    import org.apache.royale.html.supportClasses.MXMLItemRenderer;
        
        /**
         *  The StyledMXMLItemRenderer class is the base class for 
itemRenderers that are MXML-based
@@ -31,7 +31,7 @@ package org.apache.royale.core
         *  @playerversion AIR 2.6
         *  @productversion Royale 0.9.3
         */
-       public class StyledMXMLItemRenderer extends MXMLItemRenderer implements 
IClassSelectorListSupport
+       public class StyledMXMLItemRenderer extends MXMLItemRenderer implements 
IClassSelectorListSupport, IRuntimeSelectableItemRenderer
        {
                /**
                 *  constructor.
@@ -136,5 +136,55 @@ package org.apache.royale.core
             return false;
             }
         }
+
+        /************************************
+         *  IRuntimeSelectableItemRenderer
+         ************************************/
+
+        private var _selectable:Boolean = true;
+               /**
+         *  <code>true</code> if the item renderer is can be selected
+         *  false otherwise. Use to configure a renderer to be non 
+         *  selectable.
+         *  
+         *  Defaults to true
+         * 
+         *  @langversion 3.0
+         *  @playerversion Flash 10.2
+         *  @playerversion AIR 2.6
+         *  @productversion Royale 0.9.3
+         */
+               public function get selectable():Boolean
+               {
+                       return _selectable;
+               }
+
+               public function set selectable(value:Boolean):void
+               {
+                       _selectable = value;    
+               }
+
+        private var _hoverable:Boolean = true;
+               /**
+         *  <code>true</code> if the item renderer is can be hovered
+         *  false otherwise. Use to configure a renderer to be non 
+         *  hoverable.
+         *  
+         *  Defaults to true
+         * 
+         *  @langversion 3.0
+         *  @playerversion Flash 10.2
+         *  @playerversion AIR 2.6
+         *  @productversion Royale 0.9.3
+         */
+               public function get hoverable():Boolean
+               {
+                       return _hoverable;
+               }
+
+               public function set hoverable(value:Boolean):void
+               {
+                       _hoverable = value;     
+               }
        }
 }
diff --git 
a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/supportClasses/UIItemRendererBase.as
 
b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/supportClasses/UIItemRendererBase.as
index acc779c..291596a 100644
--- 
a/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/supportClasses/UIItemRendererBase.as
+++ 
b/frameworks/projects/Basic/src/main/royale/org/apache/royale/html/supportClasses/UIItemRendererBase.as
@@ -261,29 +261,6 @@ package org.apache.royale.html.supportClasses
                        updateRenderer();
                }
                
-               private var _selectable:Boolean = true;
-               /**
-         *  <code>true</code> if the item renderer is can be selected
-         *  false otherwise. Use to configure a renderer to be non 
-         *  selectable.
-         *  
-         *  Defaults to true
-         * 
-         *  @langversion 3.0
-         *  @playerversion Flash 10.2
-         *  @playerversion AIR 2.6
-         *  @productversion Royale 0.9.3
-         */
-               public function get selectable():Boolean
-               {
-                       return _selectable;
-               }
-
-               public function set selectable(value:Boolean):void
-               {
-                       _selectable = value;    
-               }
-
                private var _selected:Boolean;
                
                /**
@@ -304,29 +281,6 @@ package org.apache.royale.html.supportClasses
                        updateRenderer();
                }
                
-               private var _hoverable:Boolean = true;
-               /**
-         *  <code>true</code> if the item renderer is can be hovered
-         *  false otherwise. Use to configure a renderer to be non 
-         *  hoverable.
-         *  
-         *  Defaults to true
-         * 
-         *  @langversion 3.0
-         *  @playerversion Flash 10.2
-         *  @playerversion AIR 2.6
-         *  @productversion Royale 0.9.3
-         */
-               public function get hoverable():Boolean
-               {
-                       return _hoverable;
-               }
-
-               public function set hoverable(value:Boolean):void
-               {
-                       _hoverable = value;     
-               }
-               
                private var _down:Boolean;
                
                /**
diff --git a/frameworks/projects/Core/src/main/royale/CoreClasses.as 
b/frameworks/projects/Core/src/main/royale/CoreClasses.as
index a06c1fc..3809370 100644
--- a/frameworks/projects/Core/src/main/royale/CoreClasses.as
+++ b/frameworks/projects/Core/src/main/royale/CoreClasses.as
@@ -129,6 +129,7 @@ import org.apache.royale.events.ItemRemovedEvent; 
ItemRemovedEvent;
        {
            import org.apache.royale.core.IScrollBarModel; IScrollBarModel;
        }
+    import org.apache.royale.core.IRuntimeSelectableItemRenderer; 
IRuntimeSelectableItemRenderer;
     import org.apache.royale.core.ISelectableItemRenderer; 
ISelectableItemRenderer;
     import org.apache.royale.core.ISelectable; ISelectable;
     import org.apache.royale.core.ISelectionModel; ISelectionModel;
diff --git 
a/frameworks/projects/Core/src/main/royale/org/apache/royale/core/ISelectableItemRenderer.as
 
b/frameworks/projects/Core/src/main/royale/org/apache/royale/core/IRuntimeSelectableItemRenderer.as
similarity index 54%
copy from 
frameworks/projects/Core/src/main/royale/org/apache/royale/core/ISelectableItemRenderer.as
copy to 
frameworks/projects/Core/src/main/royale/org/apache/royale/core/IRuntimeSelectableItemRenderer.as
index 321d75b..d5df28b 100644
--- 
a/frameworks/projects/Core/src/main/royale/org/apache/royale/core/ISelectableItemRenderer.as
+++ 
b/frameworks/projects/Core/src/main/royale/org/apache/royale/core/IRuntimeSelectableItemRenderer.as
@@ -29,33 +29,9 @@ package org.apache.royale.core
      *  @playerversion AIR 2.6
      *  @productversion Royale 0.0
      */
-       public interface ISelectableItemRenderer extends IItemRenderer
+       public interface IRuntimeSelectableItemRenderer extends 
ISelectableItemRenderer
        {
-               
-        /**
-         *  The property on the data item that the item renderer
-         *  should renderer.
-         * 
-         *  @langversion 3.0
-         *  @playerversion Flash 10.2
-         *  @playerversion AIR 2.6
-         *  @productversion Royale 0.0
-         */
-               function get labelField():String;
-               function set labelField(value:String):void;
-               
-        /**
-         *  The index of the data item in the data provider.
-         * 
-         *  @langversion 3.0
-         *  @playerversion Flash 10.2
-         *  @playerversion AIR 2.6
-         *  @productversion Royale 0.0
-         */
-               function get index():int;
-               function set index(value:int):void;
-               
-        /**
+               /**
          *  <code>true</code> if the item renderer is can be selected
          *  false otherwise. Use to configure a renderer to be non 
          *  selectable.
@@ -71,18 +47,6 @@ package org.apache.royale.core
                function set selectable(value:Boolean):void;
         
         /**
-         *  <code>true</code> if the item renderer is rendering
-         *  an item that is selected.
-         * 
-         *  @langversion 3.0
-         *  @playerversion Flash 10.2
-         *  @playerversion AIR 2.6
-         *  @productversion Royale 0.0
-         */
-               function get selected():Boolean;
-               function set selected(value:Boolean):void;
-        
-        /**
          *  <code>true</code> if the item renderer is can be hovered
          *  false otherwise. Use to configure a renderer to be non 
          *  hoverable.
@@ -96,30 +60,5 @@ package org.apache.royale.core
          */
                function get hoverable():Boolean;
                function set hoverable(value:Boolean):void;
-
-        /**
-         *  <code>true</code> if the item renderer is rendering
-         *  an item that is under the mouse pointer.
-         * 
-         *  @langversion 3.0
-         *  @playerversion Flash 10.2
-         *  @playerversion AIR 2.6
-         *  @productversion Royale 0.0
-         */
-        function get hovered():Boolean;
-        function set hovered(value:Boolean):void;
-
-        /**
-         *  <code>true</code> if the item renderer is rendering
-         *  an item that is under the mouse pointer and the
-         *  mouse button is down.
-         * 
-         *  @langversion 3.0
-         *  @playerversion Flash 10.2
-         *  @playerversion AIR 2.6
-         *  @productversion Royale 0.0
-         */
-        function get down():Boolean;
-        function set down(value:Boolean):void;
        }
 }
diff --git 
a/frameworks/projects/Core/src/main/royale/org/apache/royale/core/ISelectableItemRenderer.as
 
b/frameworks/projects/Core/src/main/royale/org/apache/royale/core/ISelectableItemRenderer.as
index 321d75b..db4e67a 100644
--- 
a/frameworks/projects/Core/src/main/royale/org/apache/royale/core/ISelectableItemRenderer.as
+++ 
b/frameworks/projects/Core/src/main/royale/org/apache/royale/core/ISelectableItemRenderer.as
@@ -56,21 +56,6 @@ package org.apache.royale.core
                function set index(value:int):void;
                
         /**
-         *  <code>true</code> if the item renderer is can be selected
-         *  false otherwise. Use to configure a renderer to be non 
-         *  selectable.
-         *  
-         *  Defaults to true
-         * 
-         *  @langversion 3.0
-         *  @playerversion Flash 10.2
-         *  @playerversion AIR 2.6
-         *  @productversion Royale 0.9.3
-         */
-               function get selectable():Boolean;
-               function set selectable(value:Boolean):void;
-        
-        /**
          *  <code>true</code> if the item renderer is rendering
          *  an item that is selected.
          * 
@@ -81,21 +66,6 @@ package org.apache.royale.core
          */
                function get selected():Boolean;
                function set selected(value:Boolean):void;
-        
-        /**
-         *  <code>true</code> if the item renderer is can be hovered
-         *  false otherwise. Use to configure a renderer to be non 
-         *  hoverable.
-         *  
-         *  Defaults to true
-         * 
-         *  @langversion 3.0
-         *  @playerversion Flash 10.2
-         *  @playerversion AIR 2.6
-         *  @productversion Royale 0.9.3
-         */
-               function get hoverable():Boolean;
-               function set hoverable(value:Boolean):void;
 
         /**
          *  <code>true</code> if the item renderer is rendering
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 63920c5..e3447ba 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,21 +18,24 @@
 
////////////////////////////////////////////////////////////////////////////////
 package org.apache.royale.jewel.beads.controllers
 {
-       import org.apache.royale.core.IBeadController;
-       import org.apache.royale.core.ISelectableItemRenderer;
-       import org.apache.royale.core.IStrand;
 COMPILE::SWF {
        import org.apache.royale.events.Event;
        import org.apache.royale.events.MouseEvent;
 }
 COMPILE::JS {
+       import goog.events;
+       import goog.events.Event;
+       import goog.events.EventType;
+
+       import org.apache.royale.core.IRuntimeSelectableItemRenderer;
        import org.apache.royale.core.UIBase;
        import org.apache.royale.core.WrappedHTMLElement;
        import org.apache.royale.events.BrowserEvent;
-       import goog.events.Event;
-       import goog.events.EventType;
-    import goog.events;
+       
 }
+       import org.apache.royale.core.IBeadController;
+       import org.apache.royale.core.ISelectableItemRenderer;
+       import org.apache.royale.core.IStrand;
        import org.apache.royale.events.ItemClickedEvent;
 
        /**
@@ -203,7 +206,7 @@ COMPILE::JS {
                protected function handleMouseUp(event:BrowserEvent):void
                {
                        event.stopImmediatePropagation();
-                       var target:ISelectableItemRenderer = 
event.currentTarget as ISelectableItemRenderer;
+                       var target:IRuntimeSelectableItemRenderer = 
event.currentTarget as IRuntimeSelectableItemRenderer;
                        if (target && target.selectable)
                        {
                                var newEvent:ItemClickedEvent = new 
ItemClickedEvent("itemClicked");

Reply via email to