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

[UX/UI] Paging through items in search results should return user to top of list #226

Open
wrenaria opened this issue Nov 1, 2024 · 3 comments
Assignees
Labels
bug Something isn't working pre-release High priority features of fixes that need to be in the initial release UX / UI

Comments

@wrenaria
Copy link

wrenaria commented Nov 1, 2024

Describe the bug
Navigating to the next page of item results should return you to the top of the item list. Currently, you remain at the bottom by the pager.

Example screenshots/recording

Screen.Recording.2024-11-01.at.3.10.04.PM.mov
@wrenaria wrenaria added bug Something isn't working UX / UI labels Nov 1, 2024
jrmerz pushed a commit that referenced this issue Nov 5, 2024
* hide typeahead search in filters on first load since ui is collapsed, remove debug code

* add material/format to item page

* fix for imagelist items that have only 1 page, to not show the multi page icon on search tiles #217

* bookreader style updates #218

* update highlighted collection card to show image override #221

* update number formats for result counts to have commas #222

* add delete to admin ui cards, also fix glitch when deleting certain collections #220

* pagination/scrollTo fixes for search page and other parts of app #226 #157

* update link to collections on search to only return if > 1 results when a collection is selected/part of url #219

* fix for image size rendering on item/ark:/87293/d3cv4bw94
@UcDust
Copy link
Contributor

UcDust commented Nov 5, 2024

@wrenaria I think this should be deployed on dev now.

@wrenaria
Copy link
Author

wrenaria commented Nov 5, 2024

@UcDust Looks good on Search/Items. This behavior also needs to be applied to all other pages that have pagers on them:

jrmerz pushed a commit that referenced this issue Nov 13, 2024
* updates for mosaic layout for performance, also set selected display type on first load

* remove image background before image loads on mosiac

* update main content min height to fix jumpy slow connection rendering

* fix for download label when tif is super large, /item/ark:/87293/d32j6886n

* fix for nav state when changing between items that use bookreader vs imagelist #229

* fix for video media type #232

* fix for nav changing between item display types

* update search sort to include score and id #219, #227

* style fixes for bookreader search #218

* update homepage mobile header with consistent links #234

* add more space above item highlights heading #230

* fix collection admin ui to allow 0 highlights #231

* update pagination scroll to top of page on browse-by page #226

* margin/text updates for mobile on item page #235

* update part-of slightly after chatting with Kimmy #235

* fix sorting a-z on browse page #238

* fix space between collection heading and disclaimer #236

* bugfix for building pdf when no manifest
@UcDust
Copy link
Contributor

UcDust commented Nov 13, 2024

@wrenaria this should be deployed now on dev.

@UcDust UcDust added the pre-release High priority features of fixes that need to be in the initial release label Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working pre-release High priority features of fixes that need to be in the initial release UX / UI
Projects
None yet
Development

No branches or pull requests

2 participants