FIX: Don't show pagination when unnecessary (#138)

closes #138
dwhieb · Jul 13, 2024 · e292c99 · e292c99
1 parent fddc452
commit e292c99
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/pages/Search/Search.hbs b/pages/Search/Search.hbs
@@ -40,7 +40,9 @@
         {{> em-dash/em-dash }}
       </div>
 
-      {{> Search/components/pagination/pagination }}
+      {{#if pagination.show }}
+        {{> Search/components/pagination/pagination }}
+      {{/if}}
 
       <div aria-labelledby=results-caption class='results-wrapper scroll-shadow' role=region tabindex=0>
         <table class=results>
@@ -118,7 +120,9 @@
         </table>
       </div>
 
-      {{> Search/components/pagination/pagination }}
+      {{#if pagination.show }}
+        {{> Search/components/pagination/pagination }}
+      {{/if}}
 
       <section class='citation sticky'>
 

diff --git a/pages/Search/Search.js b/pages/Search/Search.js
@@ -107,6 +107,7 @@ export function Search(req, res) {
       },
       nextPages,
       prevPages,
+      show:       allResults.length > limit,
       startIndex: (offset + 1).toLocaleString(),
     },
     results,

diff --git a/pages/Search/components/pagination/pagination.hbs b/pages/Search/components/pagination/pagination.hbs
@@ -31,7 +31,7 @@
         <a href='{{ link }}'>{{#if jump }}…{{else}}{{ pageNum }}{{/if}}</a>
       </li>
     {{/each}}
-    {{#unless (is endIndex totalResults) }}
+    {{#unless (is endIndex ../totalResults) }}
       <li>
         <a href='{{ links.nextPage }}'>
           <span aria-hidden=true>›</span>