[paginationStore] Clamp page between valid values

techniq · Jun 21, 2024 · 6adc3f2 · 6adc3f2
1 parent cc27658
commit 6adc3f2
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/.changeset/thin-roses-fix.md b/.changeset/thin-roses-fix.md
@@ -0,0 +1,5 @@
+---
+'svelte-ux': patch
+---
+
+[paginationStore] Clamp page between valid values
diff --git a/packages/svelte-ux/src/lib/stores/paginationStore.ts b/packages/svelte-ux/src/lib/stores/paginationStore.ts
@@ -1,4 +1,5 @@
 import { writable } from 'svelte/store';
+import { clamp } from '../utils/number.js';
 
 export type PaginationProps = {
   page?: number;
@@ -40,8 +41,10 @@ export default function paginationStore(props?: PaginationProps) {
   };
 }
 
-function createState(page: number, perPage: number, total: number) {
+function createState(_page: number, perPage: number, total: number) {
   const totalPages = Math.ceil(total / perPage);
+  // Do not allow page to exceed bounds (ex. call nextPage() when on last page)
+  const page = clamp(_page, 1, totalPages);
 
   return {
     page,