[ARVADOS-WORKBENCH2] updated: 1.1.4-520-g8089fda

[Git user]

Summary of changes:
 .../data-explorer/data-explorer.tsx                | 56 ++++++++++++----------
 1 file changed, 31 insertions(+), 25 deletions(-)

       via  8089fda80143390d021de09b64588cb9ddd1e194 (commit)
      from  4a24786b1ab2199d3841226eed83be56a83645fc (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 8089fda80143390d021de09b64588cb9ddd1e194
Author: Michal Klobukowski <michal.klobukowski at contractors.roche.com>
Date:   Fri Aug 3 13:07:27 2018 +0200

    Memoize columns in data-explorer dispatch mapping function
    
    Feature #13855
    
    Arvados-DCO-1.1-Signed-off-by: Michal Klobukowski <michal.klobukowski at contractors.roche.com>

diff --git a/src/views-components/data-explorer/data-explorer.tsx b/src/views-components/data-explorer/data-explorer.tsx
index 6449bf8..e13e8af 100644
--- a/src/views-components/data-explorer/data-explorer.tsx
+++ b/src/views-components/data-explorer/data-explorer.tsx
@@ -24,40 +24,46 @@ interface Props {
 const mapStateToProps = (state: RootState, { id }: Props) =>
     getDataExplorer(state.dataExplorer, id);
 
-const mapDispatchToProps = (dispatch: Dispatch, { id, columns, onRowClick, onRowDoubleClick, onContextMenu }: Props) => {
-    dispatch(dataExplorerActions.SET_COLUMNS({ id, columns }));
-    return {
-        onSearch: (searchValue: string) => {
-            dispatch(dataExplorerActions.SET_SEARCH_VALUE({ id, searchValue }));
-        },
+const mapDispatchToProps = () => {
+    let prevColumns: DataColumns<any>;
+    return (dispatch: Dispatch, { id, columns, onRowClick, onRowDoubleClick, onContextMenu }: Props) => {
+        if (columns !== prevColumns) {
+            prevColumns = columns;
+            dispatch(dataExplorerActions.SET_COLUMNS({ id, columns }));
+        }
+        return {
+            onSearch: (searchValue: string) => {
+                dispatch(dataExplorerActions.SET_SEARCH_VALUE({ id, searchValue }));
+            },
 
-        onColumnToggle: (column: DataColumn<any>) => {
-            dispatch(dataExplorerActions.TOGGLE_COLUMN({ id, columnName: column.name }));
-        },
+            onColumnToggle: (column: DataColumn<any>) => {
+                dispatch(dataExplorerActions.TOGGLE_COLUMN({ id, columnName: column.name }));
+            },
 
-        onSortToggle: (column: DataColumn<any>) => {
-            dispatch(dataExplorerActions.TOGGLE_SORT({ id, columnName: column.name }));
-        },
+            onSortToggle: (column: DataColumn<any>) => {
+                dispatch(dataExplorerActions.TOGGLE_SORT({ id, columnName: column.name }));
+            },
 
-        onFiltersChange: (filters: DataTableFilterItem[], column: DataColumn<any>) => {
-            dispatch(dataExplorerActions.SET_FILTERS({ id, columnName: column.name, filters }));
-        },
+            onFiltersChange: (filters: DataTableFilterItem[], column: DataColumn<any>) => {
+                dispatch(dataExplorerActions.SET_FILTERS({ id, columnName: column.name, filters }));
+            },
 
-        onChangePage: (page: number) => {
-            dispatch(dataExplorerActions.SET_PAGE({ id, page }));
-        },
+            onChangePage: (page: number) => {
+                dispatch(dataExplorerActions.SET_PAGE({ id, page }));
+            },
 
-        onChangeRowsPerPage: (rowsPerPage: number) => {
-            dispatch(dataExplorerActions.SET_ROWS_PER_PAGE({ id, rowsPerPage }));
-        },
+            onChangeRowsPerPage: (rowsPerPage: number) => {
+                dispatch(dataExplorerActions.SET_ROWS_PER_PAGE({ id, rowsPerPage }));
+            },
 
-        onRowClick,
+            onRowClick,
 
-        onRowDoubleClick,
+            onRowDoubleClick,
 
-        onContextMenu,
+            onContextMenu,
+        };
     };
 };
 
-export const DataExplorer = connect(mapStateToProps, mapDispatchToProps)(DataExplorerComponent);
+export const DataExplorer = connect(mapStateToProps, mapDispatchToProps())(DataExplorerComponent);
 

-----------------------------------------------------------------------


hooks/post-receive
--