Pagers are great for browsing, but for printing it would be nice to have a link that shows everything.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | pager_show_all_4.patch | 3.23 KB | v1nce |
| #5 | pager_show_all_3.patch | 3.33 KB | v1nce |
| #4 | pager_show_all_2.patch | 2.89 KB | v1nce |
| #3 | pager_show_all_1.patch | 2.89 KB | v1nce |
Comments
Comment #1
darren ohComment #2
Gurpartap Singh commentedThe paging module checks for ?page=full query in url to show all pages or a single page. A default drupal behavior would be good to use, and easier for users to try out, since it would be a site-wide feature.
Comment #3
v1nce commentedI don't see anything in pager.inc that checks for ?page=full to show all pages.
Here's a quick patch against the d5 branch that allows for show all pagination. It will need to be polished and brought up to d7.
Comment #4
v1nce commentedUpdated patch to correctly set
$pager_current.Comment #5
v1nce commentedOne more time. Updated patch to allow proper handling of pagination list when viewing all items.
Comment #6
v1nce commentedWrong file path in above patch and removed separate feature that creeped in.
Comment #7
lilou commentedhttp://drupal.org/patch/create :
Comment #8
jody lynnI think this is a bad idea as a generic core feature as it will cause frequent problems in which there is insufficient memory to display an unlimited number of items.