- Issue created by @zengenuity
- π¬π§United Kingdom scott_euser
Thanks! 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).
- πΊπΈUnited States zengenuity
Adding 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.
- πΊπΈUnited States zengenuity
Updated the MR to use a full pager with 50 items per page.
-
scott_euser β
committed 148dfa8a on 2.1.x authored by
zengenuity β
Issue #3512538 by zengenuity, scott_euser: Default menus view hides...
-
scott_euser β
committed 148dfa8a on 2.1.x authored by
zengenuity β
- π¬π§United Kingdom scott_euser
Checked and we don't do update hooks for existing views. Added a tiny bit more test coverage to pager. Thank you!
Automatically closed - issue fixed for 2 weeks with no activity.