fix extra bulk row selection events

KevinVandy · KevinVandy · commit 87b2cc89423e · 2024-02-05T01:34:17.000-06:00
diff --git a/packages/material-react-table/src/utils/row.utils.ts b/packages/material-react-table/src/utils/row.utils.ts
@@ -184,8 +184,12 @@ export const getMRT_RowSelectionHandler =
           lastIndex < currentIndex
             ? [lastIndex, currentIndex]
             : [currentIndex, lastIndex];
-        for (let i = start; i <= end; i++) {
-          rows[i].toggleSelected(!isChecked);
+        if (
+          rows[currentIndex].getIsSelected() !== rows[lastIndex].getIsSelected()
+        ) {
+          for (let i = start; i <= end; i++) {
+            rows[i].toggleSelected(!isChecked);
+          }
         }
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -184,8 +184,12 @@ export const getMRT_RowSelectionHandler =`
`184`	`184`	`lastIndex < currentIndex`
`185`	`185`	`? [lastIndex, currentIndex]`
`186`	`186`	`: [currentIndex, lastIndex];`
`187`		`- for (let i = start; i <= end; i++) {`
`188`		`- rows[i].toggleSelected(!isChecked);`
	`187`	`+ if (`
	`188`	`+ rows[currentIndex].getIsSelected() !== rows[lastIndex].getIsSelected()`
	`189`	`+ ) {`
	`190`	`+ for (let i = start; i <= end; i++) {`
	`191`	`+ rows[i].toggleSelected(!isChecked);`
	`192`	`+ }`
`189`	`193`	`}`
`190`	`194`	`}`
`191`	`195`	`}`