fix: next button is not enabled with pageUnknown prop (#17407)

Co-authored-by: Gururaj J <89023023+Gururajj77@users.noreply.github.com>
carbon-design-system · Sep 13, 2024 · 72fb7c0 · 72fb7c0
1 parent 885a114
commit 72fb7c0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/packages/react/src/components/Pagination/Pagination-test.js b/packages/react/src/components/Pagination/Pagination-test.js
@@ -222,6 +222,11 @@ describe('Pagination', () => {
       ).not.toBeInTheDocument();
     });
 
+    it('Next button should be enabled if totalItems undefined and pagesUnknown is true', () => {
+      render(<Pagination pagesUnknown={true} pageSizes={[10]} />);
+      expect(screen.getByLabelText('Next page')).not.toBeDisabled();
+    });
+
     it('should handle zero total items', () => {
       render(
         <Pagination
@@ -231,7 +236,7 @@ describe('Pagination', () => {
           page={1}
         />
       );
-
+      expect(screen.getByLabelText('Next page')).toBeDisabled();
       expect(screen.getByText('0–0 of 0 items')).toBeInTheDocument();
     });
   });

diff --git a/packages/react/src/components/Pagination/Pagination.tsx b/packages/react/src/components/Pagination/Pagination.tsx
@@ -225,7 +225,8 @@ const Pagination = React.forwardRef(function Pagination(
     [`${prefix}--pagination__button--backward`]: true,
     [`${prefix}--pagination__button--no-index`]: backButtonDisabled,
   });
-  const forwardButtonDisabled = disabled || page === totalPages;
+  const forwardButtonDisabled =
+    disabled || (page === totalPages && !pagesUnknown);
   const forwardButtonClasses = cx({
     [`${prefix}--pagination__button`]: true,
     [`${prefix}--pagination__button--forward`]: true,