-
Notifications
You must be signed in to change notification settings - Fork 0
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
Labels
bug
Something isn't working
pre-release
High priority features of fixes that need to be in the initial release
UX / UI
Comments
UcDust
added a commit
that referenced
this issue
Nov 4, 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
@wrenaria I think this should be deployed on dev now. |
@UcDust Looks good on Search/Items. This behavior also needs to be applied to all other pages that have pagers on them: |
UcDust
added a commit
that referenced
this issue
Nov 12, 2024
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
@wrenaria this should be deployed now on dev. |
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
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
The text was updated successfully, but these errors were encountered: