[pagination] Use compact layout when counts are zero

Change-Id: I899a187b211702ec415758944ff1ed0404944470

Author: Boulea7

packages/mui-material/src/usePagination/usePagination.js

@@ -66,33 +66,39 @@ export default function usePagination(props = {}) {
     count - boundaryCount - 1,
   );
 
+  // Start ellipsis
+  const startEllipsis = [];
+  if (siblingsStart > boundaryCount + 2) {
+    startEllipsis.push('start-ellipsis');
+  } else if (boundaryCount + 1 < count - boundaryCount) {
+    startEllipsis.push(boundaryCount + 1);
+  }
+
+  // End ellipsis
+  const endEllipsis = [];
+  if (siblingsEnd < count - boundaryCount - 1) {
+    endEllipsis.push('end-ellipsis');
+  } else if (count - boundaryCount > boundaryCount) {
+    endEllipsis.push(count - boundaryCount);
+  }
+
+  const body =
+    boundaryCount === 0 && siblingCount === 0
+      ? [page]
+      : [
+          ...startPages,
+          ...startEllipsis,
+          ...range(siblingsStart, siblingsEnd),
+          ...endEllipsis,
+          ...endPages,
+        ];
+
   // Basic list of items to render
   // for example itemList = ['first', 'previous', 1, 'ellipsis', 4, 5, 6, 'ellipsis', 10, 'next', 'last']
   const itemList = [
     ...(showFirstButton ? ['first'] : []),
     ...(hidePrevButton ? [] : ['previous']),
-    ...startPages,
-
-    // Start ellipsis
-    // eslint-disable-next-line no-nested-ternary
-    ...(siblingsStart > boundaryCount + 2
-      ? ['start-ellipsis']
-      : boundaryCount + 1 < count - boundaryCount
-        ? [boundaryCount + 1]
-        : []),
-
-    // Sibling pages
-    ...range(siblingsStart, siblingsEnd),
-
-    // End ellipsis
-    // eslint-disable-next-line no-nested-ternary
-    ...(siblingsEnd < count - boundaryCount - 1
-      ? ['end-ellipsis']
-      : count - boundaryCount > boundaryCount
-        ? [count - boundaryCount]
-        : []),
-
-    ...endPages,
+    ...body,
     ...(hideNextButton ? [] : ['next']),
     ...(showLastButton ? ['last'] : []),
   ];

packages/mui-material/src/usePagination/usePagination.test.js

@@ -151,23 +151,21 @@ describe('usePagination', () => {
     expect(items[9]).to.have.property('page', 11);
   });
 
-  it('should support boundaryCount={0}', () => {
-    let items;
+  it('uses a compact layout when boundaryCount and siblingCount are zero', () => {
+    [1, 6, 11].forEach((page) => {
+      const items = renderHook(() =>
+        usePagination({ count: 11, page, boundaryCount: 0, siblingCount: 0 }),
+      ).result.current.items;
 
-    items = renderHook(() =>
-      usePagination({ count: 11, page: 6, boundaryCount: 0, siblingCount: 0 }),
-    ).result.current.items;
-    expect(serialize(items)).to.deep.equal([
-      'previous',
-      'start-ellipsis',
-      6,
-      'end-ellipsis',
-      'next',
-    ]);
+      expect(serialize(items)).to.deep.equal(['previous', page, 'next']);
+    });
+  });
 
-    items = renderHook(() =>
+  it('should support boundaryCount={0}', () => {
+    const items = renderHook(() =>
       usePagination({ count: 11, page: 6, boundaryCount: 0, siblingCount: 1 }),
     ).result.current.items;
+
     expect(serialize(items)).to.deep.equal([
       'previous',
       'start-ellipsis',