[PATCH 1/7] pagination: Fix quirks
Stephen Finucane
stephen at that.guru
Sat Jan 27 08:15:49 AEDT 2018
On Thu, 2018-01-25 at 13:43 +1100, Daniel Axtens wrote:
> There are a couple of pages where the clickable list of pages
> would include missing or duplicate pages.
>
> Write a test that ensures:
> - you always have a link to the next/prev numbered page
> - there are no duplicate page numbers
>
> Fiddle with the pagination algorithm to get it to pass - required
> tweaking a display parameter and a couple of comparison operators,
> so all pretty minor.
>
> Now, if there are 10 pages, the displayed page numbers for a given
> page are as follows:
>
> Page # | Displayed page #s
> ---------------------------
> 1 | [] [1, 2, 3, 4] [9, 10]
> 2 | [] [1, 2, 3, 4] [9, 10]
> 3 | [] [1, 2, 3, 4] [9, 10]
> 4 | [1, 2] [3, 4, 5] [9, 10]
> 5 | [1, 2] [4, 5, 6] [9, 10]
> 6 | [1, 2] [5, 6, 7] [9, 10]
> 7 | [1, 2] [6, 7, 8] [9, 10]
> 8 | [1, 2] [7, 8, 9, 10] []
> 9 | [1, 2] [7, 8, 9, 10] []
> 10 | [1, 2] [7, 8, 9, 10] []
>
> Closes: #102
> Signed-off-by: Daniel Axtens <dja at axtens.net>
Yup, looks fine to me.
Reviewed-by: Stephen Finucane <stephen at that.guru>
I'll backport this too.
More information about the Patchwork
mailing list