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 79a1aa2 Clean names and separate add item from List and Table 79a1aa2 is described below commit 79a1aa2a9ac7afdb3edc2b82435bab2be9b2bd01 Author: Carlos Rovira <carlosrov...@apache.org> AuthorDate: Thu Aug 23 20:54:53 2018 +0200 Clean names and separate add item from List and Table --- .../JewelExample/src/main/royale/ListPlayGround.mxml | 4 ++-- .../JewelExample/src/main/royale/TablePlayGround.mxml | 2 +- .../Jewel/src/main/resources/jewel-manifest.xml | 4 ++-- ...tData.as => AddListItemRendererForArrayListData.as} | 18 +++++++++--------- .../AddTableItemRendererForArrayListData.as | 12 +++++------- ...ata.as => RemoveAllItemRendererForArrayListData.as} | 18 +++++++++--------- 6 files changed, 28 insertions(+), 30 deletions(-) diff --git a/examples/royale/JewelExample/src/main/royale/ListPlayGround.mxml b/examples/royale/JewelExample/src/main/royale/ListPlayGround.mxml index ec24b0d..6b60d03 100644 --- a/examples/royale/JewelExample/src/main/royale/ListPlayGround.mxml +++ b/examples/royale/JewelExample/src/main/royale/ListPlayGround.mxml @@ -112,10 +112,10 @@ limitations under the License. sourceID="listModel" sourcePropertyName="iconListData" destinationPropertyName="dataProvider" /> - <j:DynamicAddItemRendererForArrayListData/> + <j:AddListItemRendererForArrayListData/> <j:DynamicRemoveItemRendererForArrayListData/> <j:DynamicUpdateItemRendererForArrayListData/> - <j:DynamicRemoveAllItemRendererForArrayListData/> + <j:RemoveAllItemRendererForArrayListData/> </j:beads> </j:List> <j:Button text="Add item" click="addItem()"/> diff --git a/examples/royale/JewelExample/src/main/royale/TablePlayGround.mxml b/examples/royale/JewelExample/src/main/royale/TablePlayGround.mxml index de9c76a..6dd838a 100644 --- a/examples/royale/JewelExample/src/main/royale/TablePlayGround.mxml +++ b/examples/royale/JewelExample/src/main/royale/TablePlayGround.mxml @@ -188,7 +188,7 @@ limitations under the License. <j:AddTableItemRendererForArrayListData/> <j:DynamicRemoveItemRendererForArrayListData/> <j:DynamicUpdateItemRendererForArrayListData/> - <j:DynamicRemoveAllItemRendererForArrayListData/> + <j:RemoveAllItemRendererForArrayListData/> </j:beads> </j:Table> diff --git a/frameworks/projects/Jewel/src/main/resources/jewel-manifest.xml b/frameworks/projects/Jewel/src/main/resources/jewel-manifest.xml index 7820f6f..e5691b9 100644 --- a/frameworks/projects/Jewel/src/main/resources/jewel-manifest.xml +++ b/frameworks/projects/Jewel/src/main/resources/jewel-manifest.xml @@ -120,10 +120,10 @@ <component id="TableItemRendererFactoryForCollectionView" class="org.apache.royale.jewel.beads.itemRenderers.TableItemRendererFactoryForCollectionView"/> - <component id="DynamicAddItemRendererForArrayListData" class="org.apache.royale.jewel.beads.itemRenderers.DynamicAddItemRendererForArrayListData" /> + <component id="AddListItemRendererForArrayListData" class="org.apache.royale.jewel.beads.itemRenderers.AddListItemRendererForArrayListData" /> <component id="DynamicRemoveItemRendererForArrayListData" class="org.apache.royale.jewel.beads.itemRenderers.DynamicRemoveItemRendererForArrayListData" /> <component id="DynamicUpdateItemRendererForArrayListData" class="org.apache.royale.jewel.beads.itemRenderers.DynamicUpdateItemRendererForArrayListData" /> - <component id="DynamicRemoveAllItemRendererForArrayListData" class="org.apache.royale.jewel.beads.itemRenderers.DynamicRemoveAllItemRendererForArrayListData" /> + <component id="RemoveAllItemRendererForArrayListData" class="org.apache.royale.jewel.beads.itemRenderers.RemoveAllItemRendererForArrayListData" /> <component id="AddTableItemRendererForArrayListData" class="org.apache.royale.jewel.beads.itemRenderers.AddTableItemRendererForArrayListData" /> <!-- <component id="JewelLabelViewBead" class="org.apache.royale.jewel.beads.views.JewelLabelViewBead"/> --> diff --git a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/DynamicAddItemRendererForArrayListData.as b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddListItemRendererForArrayListData.as similarity index 94% rename from frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/DynamicAddItemRendererForArrayListData.as rename to frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddListItemRendererForArrayListData.as index e311587..c4bc590 100644 --- a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/DynamicAddItemRendererForArrayListData.as +++ b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddListItemRendererForArrayListData.as @@ -35,15 +35,15 @@ package org.apache.royale.jewel.beads.itemRenderers import org.apache.royale.utils.loadBeadFromValuesManager; /** - * Handles the adding of an itemRenderer once the corresponding datum has been added + * Handles the adding of an itemRenderer in a List component once the corresponding datum has been added * from the IDataProviderModel. * * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ - public class DynamicAddItemRendererForArrayListData implements IBead + public class AddListItemRendererForArrayListData implements IBead { /** * Constructor @@ -51,9 +51,9 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ - public function DynamicAddItemRendererForArrayListData() + public function AddListItemRendererForArrayListData() { } @@ -67,7 +67,7 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ public function set strand(value:IStrand):void { @@ -115,7 +115,7 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ protected function handleItemAdded(event:CollectionEvent):void { @@ -145,7 +145,7 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ public function get dataProviderModel(): IDataProviderModel { @@ -164,7 +164,7 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ public function get itemRendererParent():IItemRendererParent { diff --git a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddTableItemRendererForArrayListData.as b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddTableItemRendererForArrayListData.as index eaee0cd..d7c29ad 100644 --- a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddTableItemRendererForArrayListData.as +++ b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddTableItemRendererForArrayListData.as @@ -47,7 +47,7 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ public class AddTableItemRendererForArrayListData implements IBead { @@ -57,7 +57,7 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ public function AddTableItemRendererForArrayListData() { @@ -73,7 +73,7 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ public function set strand(value:IStrand):void { @@ -121,7 +121,7 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ protected function handleItemAdded(event:CollectionEvent):void { @@ -131,7 +131,6 @@ package org.apache.royale.jewel.beads.itemRenderers var ir:ITextItemRenderer; var index:int = event.index * model.columns.length; - var rowIndex:int = index; for(var j:int = 0; j < model.columns.length; j++) { column = model.columns[j] as TableColumn; @@ -161,7 +160,6 @@ package org.apache.royale.jewel.beads.itemRenderers // update the index values in the itemRenderers to correspond to their shifted positions. var n:int = tbody.numElements; var d:DataItemRenderer; - index = event.index * model.columns.length; for (var i:int = event.index; i < n; i++) { d = tbody.getItemRendererForIndex(i) as DataItemRenderer; @@ -180,7 +178,7 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ public function get itemRendererParent():IItemRendererParent { diff --git a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/DynamicRemoveAllItemRendererForArrayListData.as b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/RemoveAllItemRendererForArrayListData.as similarity index 91% rename from frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/DynamicRemoveAllItemRendererForArrayListData.as rename to frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/RemoveAllItemRendererForArrayListData.as index 409ddf0..1707f4b 100644 --- a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/DynamicRemoveAllItemRendererForArrayListData.as +++ b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/RemoveAllItemRendererForArrayListData.as @@ -30,14 +30,14 @@ package org.apache.royale.jewel.beads.itemRenderers /** * Handles the removal of all itemRenderers once the all items has been removed - * from the IDataProviderModel. + * from the IDataProviderModel. This works the same for List and Table components * * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ - public class DynamicRemoveAllItemRendererForArrayListData implements IBead + public class RemoveAllItemRendererForArrayListData implements IBead { /** * Constructor @@ -45,9 +45,9 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ - public function DynamicRemoveAllItemRendererForArrayListData() + public function RemoveAllItemRendererForArrayListData() { } @@ -59,7 +59,7 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ public function set strand(value:IStrand):void { @@ -105,7 +105,7 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ protected function handleAllItemsRemoved(event:CollectionEvent):void { @@ -129,7 +129,7 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ public function get dataProviderModel(): IDataProviderModel { @@ -148,7 +148,7 @@ package org.apache.royale.jewel.beads.itemRenderers * @langversion 3.0 * @playerversion Flash 10.2 * @playerversion AIR 2.6 - * @productversion Royale 0.9.0 + * @productversion Royale 0.9.3 */ public function get itemRendererParent():IItemRendererParent {