Merge pull request #110 from astahmer/fix/page-size-default

fix: pageSize should default to pageSizes[0]
Kuechlin · Oct 28, 2022 · 1ac4ac1 · 1ac4ac1
2 parents a2a8743 + 1779c91
commit 1ac4ac1
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/DataGrid.tsx b/src/DataGrid.tsx
@@ -217,13 +217,14 @@ export function DataGrid<TData extends RowData>({
     onRowSelectionChange: handleRowSelectionChange,
   }));
 
+  const defaultPageSize = Number(pageSizes?.[0] ?? DEFAULT_INITIAL_SIZE);
   useEffect(() => {
     if (withPagination) {
-      table.setPageSize(initialState?.pagination?.pageSize ?? DEFAULT_INITIAL_SIZE);
+      table.setPageSize(initialState?.pagination?.pageSize ?? defaultPageSize);
     } else {
       table.setPageSize(data.length);
     }
-  }, [table, withPagination, data.length, initialState?.pagination?.pageSize]);
+  }, [table, withPagination, data.length, initialState?.pagination?.pageSize, defaultPageSize]);
 
   return (
     <Stack {...others} spacing={verticalSpacing} className={classes.wrapper}>