Skip to content

Commit c665f79

Browse files
committed
fix(useRowSelect) only multi select if it is defined somewhere
1 parent f269758 commit c665f79

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@table-library/react-table-library",
3-
"version": "2.0.12",
3+
"version": "2.0.13",
44
"description": "react-table-library",
55
"type": "module",
66
"main": "main.js",

src/select/useRowSelect.tsx

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,15 +60,19 @@ const getRowProps = (props: RowProps, features: Features): FeatureProps => {
6060
if (!isRowClick(event)) return;
6161
if (select.options.clickType !== SelectClickTypes.RowClick) return;
6262

63+
const hasMultiSelectType =
64+
select.options.rowSelect === SelectTypes.MultiSelect ||
65+
select.options.buttonSelect === SelectTypes.MultiSelect;
66+
6367
const isMultiSelectType = select.options.rowSelect === SelectTypes.MultiSelect;
6468

6569
// optional ways to activate multi-select with keyboard
6670
const isCommandSelectType = !!(event as any).metaKey;
6771
const isShiftSelectType = !!(event as any).shiftKey;
6872

69-
if (isCommandSelectType && isMultiSelectType) {
73+
if (isCommandSelectType && hasMultiSelectType) {
7074
select.fns.onToggleById(node.id);
71-
} else if (isShiftSelectType && isMultiSelectType) {
75+
} else if (isShiftSelectType && hasMultiSelectType) {
7276
select.fns.onToggleByIdShift(node.id, select.options, applyModifiers(features));
7377
} else if (isMultiSelectType) {
7478
select.fns.onToggleById(node.id);

0 commit comments

Comments
 (0)