Implement Ctrl-A for grid (Fixes #1149) (#1165)

xh · Jun 6, 2019 · 0b670a1 · 0b670a1
1 parent 3dbf4a2
commit 0b670a1
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 3 deletions.
diff --git a/cmp/grid/Grid.js b/cmp/grid/Grid.js
@@ -145,9 +145,8 @@ export class Grid extends Component {
     }
 
     render() {
-        const {model, props, agOptions} = this,
-            {treeMode, agGridModel} = model,
-            {onKeyDown} = props;
+        const {model, agOptions, onKeyDown} = this,
+            {treeMode, agGridModel} = model;
 
         // Note that we intentionally do *not* render the agGridReact element below with either the data
         // or the columns. These two bits are the most volatile in our GridModel, and this causes
@@ -597,6 +596,15 @@ export class Grid extends Component {
         return column.isTreeColumn ? node.data[column.field] : value;
     }
 
+    onKeyDown = (evt) => {
+        const {selModel} = this.model;
+        if ((evt.ctrlKey || evt.metaKey) && evt.key == 'a' && selModel.mode === 'multiple') {
+            selModel.selectAll();
+            return;
+        }
+
+        if (this.props.onKeyDown) this.props.onKeyDown(evt);
+    }
 }
 
 export const grid = elemFactory(Grid);
diff --git a/data/StoreSelectionModel.js b/data/StoreSelectionModel.js
@@ -84,6 +84,15 @@ export class StoreSelectionModel {
         this.ids = clearSelection ? ids : union(this.ids, ids);
     }
 
+    /** Select all filtered records. */
+    @action
+    selectAll() {
+        if (this.mode === 'multiple') {
+            this.select(this.store.records);
+        }
+    }
+
+
     /** Clear the selection. */
     @action
     clear() {