Stop Showing Pagination Buttons When There is only one page

[SharkyBytes]

Problem

This issue of hiding the pager when there is only one page was proposed by @RayBB. I am raising the issue on his behalf.

I also believe that if certain buttons are unclickable, we should stop giving false indications to users. For example, when we are already on first page, there's no use in showing the 'First' and 'Previous' buttons, as they serve no purpose. It's better to avoid showing these buttons.

What do you think about this, @RayBB, @mekarpeles?

Reproducing the bug

1. Go to ... Any of the Author's works -> View all Editions (https://openlibrary.org/works/OL20929W/The_Spanish_gypsy?mode=all)

• Expected behavior:

• Actual behavior:

Context

• Browser (Chrome, Safari, Firefox, etc):
• OS (Windows, Mac, etc):
• Logged in (Y/N):
• Environment (prod, dev, local): prod

Breakdown

Requirements Checklist

• [ ]

Related files

• Related Slack discussion

Stakeholders

• @RayBB

---

Instructions for Contributors

• Please run these commands to ensure your repository is up to date before creating a new branch to work on this issue and each time after pushing code to Github, because the pre-commit bot may add commits to your PRs upstream.

[RayBB]

@SharkyBytes this looks good. I assigned you and you can go ahead and work on it.
We want to hide the pager if there is only one page.

[SharkyBytes]

Hi @RayBB, @mekarpeles & @cdrini ,

I’ve implemented the fix you mentioned, Ray, and I wanted to share some updates along with a suggestion I’ve worked on. I’ve addressed the issue as per your request, but I also made an additional improvement that I believe aligns with how pagination is handled elsewhere in the application (e.g., on the Trending page Video).

To give you a clear view, I’ve recorded two versions for comparison:

->The implementation based on your suggestion
Here’s the version with the fix you asked for:
Video Link

->The version with my additional suggestion
In this version, I’ve implemented your fix as well as a behavior where navigation buttons (e.g., "Previous" and "First") are hidden when they aren’t needed on the first page, and similarly, "Next" and "Last" are hidden on the last page. This approach is consistent with how pagination works on the trending page.
Video Link

Could you please have a look and let me know your thoughts? I’d love to hear your feedback on both implementations and understand which one you feel works best.