Improve the JS pagination to handle the case when the `X-Total` and `X-Total-Pages` headers aren't present
requested to merge 57991-frontend-pagination-needs-to-handle-cases-where-the-x-total-pages-header-isn-t-present into master
What does this MR do?
This improves the JS pagination to handle the case when the X-Total and X-Total-Pages headers aren't present.
Screenshots
With the X-Total and X-Total-Pages present
| Page | Screenshot | Notes |
|---|---|---|
| First and only page |  |
No pagination. |
| First page |  |
"Next", page numbers and "Last" are enabled. "Prev" is disabled. "First" isn't shown. |
| Second page |  |
"First", "Prev", page numbers, "Next" and "Last" are enabled. |
| Last page |  |
"First", "Prev" and page numbers are enabled. "Next" is disabled. "Last" isn't shown. |
With the X-Total and X-Total-Pages missing
| Page | Screenshot | Notes |
|---|---|---|
| First and only page | |
No pagination. |
| First page |  |
"Next" is enabled. "Prev" is disabled. "First", page numbers and "Last" aren't shown. |
| Second page |  |
"First", "Prev" and "Next" are enabled. Page numbers and "Last" aren't shown. |
| Last page |  |
"First" and "Previous" are enabled. "Next" is disabled. Page numbers and "Last" aren't shown. |
What are the relevant issue numbers?
Closes #57991 (closed).
Does this MR meet the acceptance criteria?
-
Changelog entry added, if necessary -
Documentation created/updated via this MR -
Documentation reviewed by technical writer or follow-up review issue created -
Tests added for this feature/bug -
Tested in all supported browsers -
Conforms to the code review guidelines -
Conforms to the merge request performance guidelines -
Conforms to the style guides -
Conforms to the database guides -
Security reports checked/validated by reviewer
Edited by Rémy Coutable