LWS-313: Yet another pagination bugfix

[jesperengstrom]

Description

Tickets involved

LWS-131

Solves

Paginating the relation search does not work in some cases.

Because: simply appending _offset to the link causes a new double-fetch (introduced with the predicate tabs) that ultimately uses the predicate view to get the data. That link excludes any applied _offset, fetching the first 10 hits again.

By using a backend-made link as template for all pagination links, we avoid the problem (and double fetch) since they include the _p param.

Summary of changes

• Fix pagination links
• Tweak pagination template
• Mobile look tweaks

[olovy]

LGTM!

[johanbissemattsson]

There is still some reactivity issues regarding the number of pagination items.

To reproduce:

1. Go to for example http://localhost:5173/find?_i=astrid&_q=astrid+hasInstanceType:Tactile&_limit=10
2. Add filters so there is few pages (but still more than one) in the pagination list (e.g. try adding the subject Vardagsliv).
3. Do a new search in the site header. The old number of visible pagination sequences are still visible (try refreshing and the number of visible sequences will change).

[jesperengstrom]

There is still some reactivity issues regarding the number of pagination items.

Good call. sequenceSize being limited due to fewer hits was reactive, but not the reverse; when number of pages increased again.

Extended the sequence to 8, let's see what happens when moving on to the styling issues...

[jesperengstrom]

Added:

• previous from api that somehow got lost
• Fixed remaining reactivity issues, should really be ok now ;)
• Center pageSequence around currentPage

Suggest we save mobile improvements for later