Skip to content

Commit ebc5bdb

Browse files
committed
fix(select) only shift select if multi select is active
1 parent 3b9d01d commit ebc5bdb

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
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.10",
3+
"version": "2.0.11",
44
"description": "react-table-library",
55
"type": "module",
66
"main": "main.js",

src/select/CellSelect.tsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -29,17 +29,17 @@ export const CellSelect: React.FC<CellSelectProps> = React.memo(
2929
: select.state.ids.includes(item.id);
3030

3131
const handleChange = React.useCallback(() => {
32-
const isSingleSelectType = select.options.buttonSelect === SelectTypes.SingleSelect;
32+
const isMuiltiSelectType = select.options.buttonSelect === SelectTypes.MultiSelect;
3333

34-
if (isShiftDown) {
34+
if (isShiftDown && isMuiltiSelectType) {
3535
select.fns.onToggleByIdShift(item.id, select.options, applyModifiers(features));
36-
} else if (isSingleSelectType) {
37-
select.fns.onToggleByIdExclusively(item.id);
38-
} /* isMtuliSelectType */ else {
36+
} else if (isMuiltiSelectType) {
3937
select.fns.onToggleByIdRecursively(item.id, {
4038
isCarryForward: select.options.isCarryForward,
4139
isPartialToAll: select.options.isPartialToAll,
4240
});
41+
} /* isSingleSelectType */ else {
42+
select.fns.onToggleByIdExclusively(item.id);
4343
}
4444
}, [isShiftDown, features, item.id, select]);
4545

src/select/useRowSelect.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,9 +66,9 @@ const getRowProps = (props: RowProps, features: Features): FeatureProps => {
6666
const isCommandSelectType = !!(event as any).metaKey;
6767
const isShiftSelectType = !!(event as any).shiftKey;
6868

69-
if (isCommandSelectType) {
69+
if (isCommandSelectType && isMultiSelectType) {
7070
select.fns.onToggleById(node.id);
71-
} else if (isShiftSelectType) {
71+
} else if (isShiftSelectType && isMultiSelectType) {
7272
select.fns.onToggleByIdShift(node.id, select.options, applyModifiers(features));
7373
} else if (isMultiSelectType) {
7474
select.fns.onToggleById(node.id);

0 commit comments

Comments
 (0)