Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bookmarks pagination has the count inside the next button #4624

Closed
sandbergja opened this issue Dec 9, 2024 · 1 comment
Closed

Bookmarks pagination has the count inside the next button #4624

sandbergja opened this issue Dec 9, 2024 · 1 comment
Assignees
Labels
bug 🐛 The application does not work as expected because of a defect

Comments

@sandbergja
Copy link
Member

Expected behavior

When I have 40 bookmarks and go to the second page of bookmarks, the next button text says "Next"

Actual behavior

The button says "Next -> 21 - 40 of 40"

the pagination at the top of the bookmarks page.  The previous button just says Previous, but the Next button is greyed out and says Next arrow 21 - 40 of 40

Steps to replicate

In firefox:

  1. Go to catalog.princeton.edu
  2. Do a search
  3. Bookmark all in the first page of search results
  4. Go to the second page of search results
  5. Bookmark all on the second page
  6. Confirm that you have 40 bookmarks
  7. Go to your bookmarks
  8. Go to the second page
  9. Look at the text of the Next button

Impact of this bug

Looks very odd

@sandbergja sandbergja added the bug 🐛 The application does not work as expected because of a defect label Dec 9, 2024
@christinach christinach self-assigned this Jan 13, 2025
@christinach
Copy link
Member

This doesn't seem to be an issue anymore:
Screenshot from Bookmarks pagination

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 The application does not work as expected because of a defect
Projects
None yet
Development

No branches or pull requests

2 participants