Clarify pagination links.
authorIain Patterson <me@iain.cx>
Mon, 4 Apr 2016 15:11:29 +0000 (11:11 -0400)
committerIain Patterson <me@iain.cx>
Mon, 4 Apr 2016 15:11:29 +0000 (11:11 -0400)
We were showing an extra page link in paginated results.
We now show ... if there are more pages whose links are not shown.

lib/functions.php

index 2c0fdea..34b23c7 100644 (file)
     list($first_page, $per_page) = pagination();
 
     $pages = ceil($pager->getNbResults() / $per_page);
-    $pages++;
+
+    /* Highlight the fact we skipped some pages. */
+    $linked_pages = $pager->getLinks($n);
+    $first_link = $linked_pages[0];
+    $last_link = end($linked_pages);
 
     $links = array();
     $links[] = page_link('First', 1, $first_page, $pages, $per_page);
     $links[] = page_link('Previous', $first_page - 1, $first_page, $pages, $per_page);
-    foreach ($pager->getLinks($n) as $link) $links[] = page_link($link, $link, $first_page, $pages, $per_page);
+    if ($first_link > 1) $links[] = page_link('...', $first_page, $pages, $per_page);
+    foreach ($linked_pages as $link) $links[] = page_link($link, $link, $first_page, $pages, $per_page);
+    if ($last_link < $pages) $links[] = page_link('...', $first_page, $pages, $per_page);
     $links[] = page_link('Next', $first_page + 1, $first_page, $pages, $per_page);
     $links[] = page_link('Last', $pages, $first_page, $pages, $per_page);