dspace-angular icon indicating copy to clipboard operation
dspace-angular copied to clipboard

Display more than 5 e-people or groups per page

Open alawvt opened this issue 2 years ago • 5 comments

Is your feature request related to a problem? Please describe. Display more that 5 e-people or groups per page. For common names it's helpful to see more choices simultaneously.

Describe the solution you'd like DSpace 6 XMLUI shows 15 per page. 20 might be better.

Describe alternatives or workarounds you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here.

alawvt avatar Aug 11 '22 20:08 alawvt

The sizes of a number of list displays should be configurable.

mwoodiupui avatar Aug 12 '22 13:08 mwoodiupui

@mwoodiupui, does that "should" mean "they should be configurable but they are not currently" or "they should already be configurable"? Thanks.

alawvt avatar Aug 12 '22 13:08 alawvt

@alawvt Sorry, my comment was unclear. The sizes of a number of list displays should be made configurable. This issue is one example. Maybe we should all put our heads together and make a list. Locally I've been asked to resize the list of results from a search, and that too is hard-coded, at least in 7.3.

mwoodiupui avatar Aug 15 '22 15:08 mwoodiupui

@mwoodiupui, thanks for the clarification. I would be happy to help make such a list. I'm still testing my way through DSpace 7 Demo.

alawvt avatar Aug 15 '22 15:08 alawvt

Agreed, this should be configurable. It appears this is hardcoded in two locations:

  • For EPeople registry listing, the page size is hardcoded here: https://github.com/DSpace/dspace-angular/blob/main/src/app/access-control/epeople-registry/epeople-registry.component.ts#L62
  • For Group registry listing, the page size is hardcoded here: https://github.com/DSpace/dspace-angular/blob/main/src/app/access-control/group-registry/groups-registry.component.ts#L57

Both could easily be made configurable in a manner similar to #1771 .

Moving this over to the 7.4 board. For now, I'm flagging this as medium priority as this page only is usable to full admins (so few are impacted), but it might be fixed alongside other similar issues like #1749 and #1750

tdonohue avatar Aug 26 '22 18:08 tdonohue