admin/twenty
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix: table deselect doesn't work with few selected records (#8692)

Browse Source 
Fixes: #8665 

[Screencast from 2024-11-23
00-33-14.webm](https://github.com/user-attachments/assets/e3b86955-4e12-4d10-9c13-ee60d7695cb3)

Co-authored-by: Félix Malfait <felix@twenty.com>
This commit is contained in:
Harsh Singh
2024-11-24 22:29:49 +05:30
committed by GitHub
parent 00791c3cd5
commit a19349bdae
1 changed files with 7 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
packages/twenty-front/src/modules/object-record/record-table/record-table-header/components/RecordTableHeaderCheckboxColumn.tsx
View File

@ -26,16 +26,17 @@ export const RecordTableHeaderCheckboxColumn = () => {
); );
const { selectAllRows, resetTableRowSelection, setHasUserSelectedAllRows } = const { selectAllRows, resetTableRowSelection, setHasUserSelectedAllRows } =
useRecordTable(); useRecordTable();
const checked = allRowsSelectedStatus === 'all'; const checked =
allRowsSelectedStatus === 'all' || allRowsSelectedStatus === 'some';
const indeterminate = allRowsSelectedStatus === 'some'; const indeterminate = allRowsSelectedStatus === 'some';
const onChange = (e: React.ChangeEvent<HTMLInputElement>) => { const onChange = () => {
if (e.target.checked) { if (checked) {
setHasUserSelectedAllRows(true);
selectAllRows();
} else {
setHasUserSelectedAllRows(false); setHasUserSelectedAllRows(false);
resetTableRowSelection(); resetTableRowSelection();
} else {
setHasUserSelectedAllRows(true);
selectAllRows();
} }
}; };