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

Fix facet prev/next pagination nav display within and outside of moda… #3548

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

seanaery
Copy link
Contributor

@seanaery seanaery commented Mar 28, 2025

…ls. Fixes #3524

  • render pagination & sort links near the top so scrolling isn't necessary to see/click them
  • style the top & bottom nav consistently
  • ensure these elements wrap appropriately in mobile/narrow viewports
  • improve display of facet browse UI when viewed on a facet page (as opposed to modal)

These revisions are primarily in response to findings in this 2024 usability evaluation (esp. see Facet Pop-Ups):
https://crkn-rcdr.gitbook.io/user-centered-design/study-blacklight-search-ux

…ls. Fixes #3524

- render pagination & sort links near the top so scrolling isn't necessary to see/click them
- style the top & bottom nav consistently
- ensure these elements wrap appropriately in mobile/narrow viewports
- improve display of facet browse UI when viewed on a facet page (as opposed to modal)
@seanaery
Copy link
Contributor Author

Screenshots

Facet in Modal (default):
Screenshot 2025-03-28 at 9 53 49 AM

Facet in Modal after Paging (See Top/Bottom Nav):
Screenshot 2025-03-28 at 9 54 17 AM

Facet in Modal w/A-Z Sort & Index Range Nav:
Screenshot 2025-03-28 at 9 54 34 AM

Facet Browse Page (note this UI becomes more useful as a destination you could link to):
Screenshot 2025-03-28 at 9 59 55 AM

Facet Browse Page w/A-Z Sort & Index Range Nav:
Screenshot 2025-03-28 at 10 00 06 AM

Mobile View, Facet Modal:
Screenshot 2025-03-28 at 9 56 03 AM

Mobile View, Facet Modal w/A-Z Sort & Index Range Nav:
Screenshot 2025-03-28 at 9 56 15 AM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Modal facet nav has odd pagination & sort above the header
1 participant