Skip to content

Commit

Permalink
fix(Selection): Change api for getSelectAllState()
Browse files Browse the repository at this point in the history
It was taking a grid argument despite not needing one, and the
documentation was out of sync.

Fixes #2086
  • Loading branch information
c0bra committed Nov 13, 2014
1 parent fa352e7 commit 5ffefaf
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions src/features/selection/js/selection.js
Original file line number Diff line number Diff line change
Expand Up @@ -240,9 +240,8 @@
* grid doesn't automatically select rows when you add extra data - so when you add data
* you need to explicitly check whether the selectAll is set, and then call setVisible rows
* if it is
* @param {bool} modifierKeysToMultiSelect true to only allow multiple rows when using ctrlKey or shiftKey is used
*/
getSelectAllState: function (grid) {
getSelectAllState: function () {
return grid.selection.selectAll;
}

Expand Down

0 comments on commit 5ffefaf

Please sign in to comment.