| Summary: | Subfunds in acqui-home.pl and aqbudgets.pl are not collapsible beyond 20th line | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Andrew Fuerste-Henry <andrew> |
| Component: | Acquisitions | Assignee: | Jonathan Druart <jonathan.druart> |
| Status: | Pushed to oldstable --- | QA Contact: | Andrew Fuerste-Henry <andrew> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | fridolin.somers, jonathan.druart, laura, lucas, michaela.sieber, nick, paul.derscheid |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: |
25.11.00,25.05.06,24.11.11
|
|
| Circulation function: | |||
| Attachments: |
Bug 40988: Remove pagination for funds on acqui home
Bug 40988: Remove pagination for funds on acqui home Bug 40988: Remove pagination for funds on acqui home |
||
|
Description
Andrew Fuerste-Henry
2025-10-09 18:11:58 UTC
Created attachment 188199 [details] [review] Bug 40988: Remove pagination for funds on acqui home As well as the fund list view (admin/aqbudgets.pl) I've tried many things to fix this problem, even using another tree table plugin (https://github.com/reside-ic/TreeTables - which works great btw). But finally noticed that we might now want the pagination at all? Can you confirm? If we remove pagination then the expand/collapse icons work for all budgets/funds. (In reply to Jonathan Druart from comment #2) > I've tried many things to fix this problem, even using another tree table > plugin (https://github.com/reside-ic/TreeTables - which works great btw). > But finally noticed that we might now want the pagination at all? Can you > confirm? > > If we remove pagination then the expand/collapse icons work for all > budgets/funds. Removing pagination would be great for libraries with a moderate number of funds, but would result in slow pages loads for libraries with a lot of funds and orders. Our preferred course here would be to get collapse/expand functionality working across paginated funds and then file a new bug to get this table to allow configuration of how many rows to show by default. It's currently paginate client-side, this patch shouldn't impact performance significantly. Created attachment 188592 [details] [review] Bug 40988: Remove pagination for funds on acqui home As well as the fund list view (admin/aqbudgets.pl) Signed-off-by: Sarah Berry <sberry@pascolibraries.org> Created attachment 188593 [details] [review] Bug 40988: Remove pagination for funds on acqui home As well as the fund list view (admin/aqbudgets.pl) Signed-off-by: Sarah Berry <sberry@pascolibraries.org> Signed-off-by: Andrew Fuerste Henry <andrew@bywatersolutions.com> Nice work everyone! Pushed to main for 25.11 Nice work everyone! Pushed to 25.05.x Pushed to 24.11.x for 24.11.11 |