Closed (fixed)
Project:
Configuration Views
Version:
2.1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
12 Mar 2025 at 17:27 UTC
Updated:
27 Mar 2025 at 05:49 UTC
Jump to comment: Most recent
Comments
Comment #3
zengenuity commentedComment #4
scott_euser commentedThanks! Yeah that makes sense to have a default for future installs with no pagination. If someone uses a microsite builder type approach that generates tons of menus, they could change it for their edge cases.
Will do a manual test run when I can + check if we normally do update hooks for unchanged views or not or only consider future installs (bit new to helping co-maintain this one).
Comment #5
zengenuity commentedAdding a pager would also be a reasonable solution, which would account for the microsite sites. If the pager is set to 50 per page or so, then most people would never see it.
Comment #6
scott_euser commentedGood compromise solution yeah
Comment #7
zengenuity commentedUpdated the MR to use a full pager with 50 items per page.
Comment #9
scott_euser commentedChecked and we don't do update hooks for existing views. Added a tiny bit more test coverage to pager. Thank you!