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 3bbb1e3  jewel-datagrid: avoid relayout when removing or adding an item
3bbb1e3 is described below

commit 3bbb1e336e0d4076654c5b5b93d72fa6cb552c6c
Author: Carlos Rovira <carlosrov...@apache.org>
AuthorDate: Wed Jan 15 18:35:42 2020 +0100

    jewel-datagrid: avoid relayout when removing or adding an item
---
 .../beads/itemRenderers/AddDataGridItemRendererForArrayListData.as    | 2 +-
 .../beads/itemRenderers/RemoveDataGridItemRendererForArrayListData.as | 2 +-
 .../main/royale/org/apache/royale/jewel/beads/views/DataGridView.as   | 4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddDataGridItemRendererForArrayListData.as
 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddDataGridItemRendererForArrayListData.as
index bb54381..c36fe03 100644
--- 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddDataGridItemRendererForArrayListData.as
+++ 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/AddDataGridItemRendererForArrayListData.as
@@ -135,7 +135,7 @@ package org.apache.royale.jewel.beads.itemRenderers
                                
ISelectionModel(_dataProviderModel).selectedIndex = 
ISelectionModel(_dataProviderModel).selectedIndex + 1;
                        }
 
-                       (_strand as IEventDispatcher).dispatchEvent(new 
Event("layoutNeeded"));
+                       // (_strand as IEventDispatcher).dispatchEvent(new 
Event("layoutNeeded"));
                }
 
                private var _dataProviderModel: IDataProviderModel;
diff --git 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/RemoveDataGridItemRendererForArrayListData.as
 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/RemoveDataGridItemRendererForArrayListData.as
index 01f40e6..d0a4a38 100644
--- 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/RemoveDataGridItemRendererForArrayListData.as
+++ 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/itemRenderers/RemoveDataGridItemRendererForArrayListData.as
@@ -112,7 +112,7 @@ package org.apache.royale.jewel.beads.itemRenderers
                                return;
 
                        // listen for individual items being removed in the 
future.
-                       dp.addEventListener(CollectionEvent.ITEM_REMOVED, 
handleItemRemoved);
+                       // dp.addEventListener(CollectionEvent.ITEM_REMOVED, 
handleItemRemoved);
                }
 
                /**
diff --git 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/views/DataGridView.as
 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/views/DataGridView.as
index 5f16622..224ca1d 100644
--- 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/views/DataGridView.as
+++ 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/views/DataGridView.as
@@ -224,7 +224,7 @@ package org.apache.royale.jewel.beads.views
                                
dp.addEventListener(CollectionEvent.ALL_ITEMS_REMOVED, 
handleItemAddedAndRemoved);
             }
 
-            host.dispatchEvent(new Event("layoutNeeded"));
+            // host.dispatchEvent(new Event("layoutNeeded"));
         }
         
         protected function updateColumnLayout(list:IDataGridColumnList):void
@@ -241,7 +241,7 @@ package org.apache.royale.jewel.beads.views
                 _listArea.positioner.style.overflow = null;
             }
             }
-            //host.dispatchEvent(new Event("layoutNeeded"));
+            // host.dispatchEvent(new Event("layoutNeeded"));
         }
 
         /**

Reply via email to