Skip to content

Commit 7efaffa

Browse files
authored
Uncheck 'Select all' header cell on page change (#1196)
* Added state variable for unchecking the select all header cell on page change * Fixed linting error * Removed state variable and used forceDeselectAll method instead * Force deselection on change of page size
1 parent fa48c7a commit 7efaffa

File tree

1 file changed

+13
-3
lines changed

1 file changed

+13
-3
lines changed

src/components/shared/Table.tsx

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,7 @@ const Table = ({
113113
};
114114

115115
const changePageSize = (size: number) => {
116+
forceDeselectAll();
116117
dispatch(updatePageSize(size));
117118
dispatch(setOffset(0));
118119
dispatch(updatePages());
@@ -331,7 +332,10 @@ const Table = ({
331332
<div className="pagination">
332333
<ButtonLikeAnchor
333334
extraClassName={cn("prev", { disabled: !isNavigatePrevious() })}
334-
onClick={() => dispatch(goToPage(pageOffset - 1))}
335+
onClick={() => {
336+
dispatch(goToPage(pageOffset - 1));
337+
forceDeselectAll();
338+
}}
335339
>
336340
<span className="sr-only">{t("TABLE_PREVIOUS")}</span>
337341
</ButtonLikeAnchor>
@@ -341,15 +345,21 @@ const Table = ({
341345
{page.label}
342346
</ButtonLikeAnchor>
343347
) : (
344-
<ButtonLikeAnchor key={key} onClick={() => dispatch(goToPage(page.number))}>
348+
<ButtonLikeAnchor key={key} onClick={() => {
349+
dispatch(goToPage(page.number));
350+
forceDeselectAll();
351+
}}>
345352
{page.label}
346353
</ButtonLikeAnchor>
347354
)
348355
)}
349356

350357
<ButtonLikeAnchor
351358
extraClassName={cn("next", { disabled: !isNavigateNext() })}
352-
onClick={() => dispatch(goToPage(pageOffset + 1))}
359+
onClick={() => {
360+
dispatch(goToPage(pageOffset + 1));
361+
forceDeselectAll();
362+
}}
353363
>
354364
<span className="sr-only">{t("TABLE_NEXT")}</span>
355365
</ButtonLikeAnchor>

0 commit comments

Comments
 (0)