refactor: simplify Pagination logic and adjust default page size in ScorecardList

- Removed redundant page size options from `Pagination` to streamline the component.
- Updated default `pageSize` from 8 to 10 for consistency across hooks and pagination logic.
- Added safeguard in `onPageChange` to prevent navigating beyond available pages.

Author: nnkogift

src/modules/ScorecardList/components/ScorecardListArea.tsx

@@ -64,14 +64,7 @@ export function ScorecardListArea() {
 				</div>
 			</Suspense>
 			<div className="p-16 w-full">
-				<Pagination
-					className="!w-full !max-w-full"
-					disabled={pager.pageCount === 1 || isEmpty(scorecards)}
-					{...pager}
-					pageSizes={Array.from(new Array(10).keys()).map((i) =>
-						((i + 1) * 4).toString()
-					)}
-				/>
+				<Pagination className="!w-full !max-w-full" {...pager} />
 			</div>
 		</div>
 	);

src/modules/ScorecardList/hooks/data.ts

@@ -66,7 +66,7 @@ export function useScorecardListData() {
 		useDataQuery<ListDataQueryResponse>(query, {
 			variables: {
 				page: searchParams.get("page") ?? 1,
-				pageSize: searchParams.get("pageSize") ?? 8,
+				pageSize: searchParams.get("pageSize") ?? 10,
 				keyword: searchParams.get("query"),
 			},
 			lazy: true,
@@ -80,7 +80,7 @@ export function useScorecardListData() {
 		refetch({
 			keyword,
 			page: 1,
-			pageSize: 8,
+			pageSize: 10,
 		});
 	}, 1000);
 
@@ -93,13 +93,16 @@ export function useScorecardListData() {
 				parseInt(searchParams.get("page") ?? "1"),
 			pageSize:
 				data?.list?.pager?.pageSize ??
-				parseInt(searchParams.get("pageSize") ?? "8"),
-			pageCount: Math.ceil(total / (data?.list?.pager?.pageSize ?? 8)),
+				parseInt(searchParams.get("pageSize") ?? "10"),
+			pageCount: Math.ceil(total / (data?.list?.pager?.pageSize ?? 10)),
 			total,
 		};
 	}, [data]);
 
 	const onPageChange = (page: number) => {
+		if (page > pager.pageCount) {
+			return;
+		}
 		setSearchParams((prev) => {
 			const updatedParams = new URLSearchParams(prev);
 			updatedParams.set("page", page.toString());
@@ -119,7 +122,7 @@ export function useScorecardListData() {
 		refetch({
 			keyword: searchParams.get("query"),
 			page: searchParams.get("page") ?? 1,
-			pageSize: searchParams.get("pageSize") ?? 8,
+			pageSize: searchParams.get("pageSize") ?? 10,
 		});
 	}, [searchParams.get("page"), searchParams.get("pageSize")]);