-
Notifications
You must be signed in to change notification settings - Fork 508
[dspace-7_x] Add paginated vocabulary search with 'Show previous/next results' controls in hierarchical browse #4504
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
base: dspace-7_x
Are you sure you want to change the base?
Conversation
…lts' controls in hierarchical browse
|
@ZahraaChreim-Atmire : Could you please create a |
|
|
Hi @ZahraaChreim-Atmire, |
…hical-browse-pagination-7_x
…2284_fix-issue4500-hierarchical-browse-pagination-7_x'
…ubject category page
References
Description
Added paging state management to the vocabulary treeview service, updating after each query based on whether previous or next pages exist; this state is exposed to the component to conditionally render navigation buttons in the hierarchical vocabulary browse.
Instructions for Reviewers
List of changes in this PR:
currentPage,totalPages,showNextPageSubject, andshowPreviousPageSubjectto thevocabulary-treeview.serviceto manage and expose pagination state.searchByQueryAndPage(query, selectedItems, page)to support page-specific vocabulary search requests. The existingsearchByQuerymethod was updated to call this with page1by default.searchByQueryAndPageto extract pagination info from the search response and update theshowNextPageSubjectandshowPreviousPageSubjectbased on whether more pages are available.showNextPage$andshowPreviousPage$in the component, derived fromshowNextPageSubject, andshowPreviousPageSubject, to reactively control the display of pagination buttons.loadNextPageand `loadPreviousPage to trigger paging behavior.To test this PR:
label="label1"throughlabel="label100"and corresponding values like"value test 1"to"value test 100"./browse/srsc).label.Expected behavior after applying this PR:
First page:

Intermediate page:

Last page:

Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You do not need to complete this checklist prior creating your PR (draft PRs are always welcome).
However, reviewers may request that you complete any actions in this list if you have not done so. If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!
mainbranch of code (unless it is a backport or is fixing an issue specific to an older branch).npm run lintnpm run check-circ-deps)package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.