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

sub-collection-list and sub-community-list components don't use communityList.pageSize as indicated in docblocks #2592

Open
ddean4040 opened this issue Oct 30, 2023 · 3 comments
Labels
bug help wanted Needs a volunteer to claim to move forward low priority

Comments

@ddean4040
Copy link

Describe the bug
The docblocks for pageSize in community-page-sub-collection-list.component.ts and community-page-sub-community-list.component.ts indicate that the default value comes from communityList.pageSize.

Changing communityList.pageSize has no effect on the page size on these pages.

To Reproduce
Steps to reproduce the behavior:

  1. Change the value of communityList.pageSize in the site config (env or config.yml)
  2. Check a community page with subcommunities and subcollections - the default page size is not changed

Expected behavior
Default page size should change

@ddean4040 ddean4040 added bug needs triage New issue needs triage and/or scheduling labels Oct 30, 2023
@github-project-automation github-project-automation bot moved this to 🆕 Triage in DSpace Backlog Oct 30, 2023
@tdonohue
Copy link
Member

@ddean4040 : I'll admit, I'm not sure what those comments are even saying. I don't know whether they are implying that communityList.pageSize should work, or if they are saying it should NOT work because the component always overrides that config.

Regardless, I agree it's odd behavior. I'll leave this open and move to our maintenance board in search of a volunteer.

That said, I'm not sure if the solution should be for the communityList.pageSize to control these page sizes, as that configuration was created to control the /community-list page (e.g. https://demo.dspace.org/community-list). Instead, we might need to determine whether there should be a similar pageSize element on the Community Homepage.

@tdonohue tdonohue added help wanted Needs a volunteer to claim to move forward low priority and removed needs triage New issue needs triage and/or scheduling labels Oct 30, 2023
@ddean4040
Copy link
Author

@tdonohue - Fair enough and thanks for looking into it :)

@alanorth
Copy link
Contributor

alanorth commented Dec 4, 2023

As of DSpace 7.6 we currently have:

  • browseBy.pageSize
  • communityList.pageSize
  • homePage.recentSubmissions.pageSize
  • homePage.topLevelCommunityList.pageSize

I don't know what the settings should be called, but I think having a configurable page size for sub-community and sub-collection lists is reasonable.

Note: I see that the custom theme themes the communityPage component as well as the community-page-sub-community-list and community-page-sub-collection-list components. It should be possible to override the pageSize then...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug help wanted Needs a volunteer to claim to move forward low priority
Projects
Development

No branches or pull requests

3 participants