Pagers are great for browsing, but for printing it would be nice to have a link that shows everything.

CommentFileSizeAuthor
#6 pager_show_all_4.patch3.23 KBv1nce
#5 pager_show_all_3.patch3.33 KBv1nce
#4 pager_show_all_2.patch2.89 KBv1nce
#3 pager_show_all_1.patch2.89 KBv1nce

Comments

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Category: support » feature
Gurpartap Singh’s picture

Gurpartap Singh commented

The 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.

v1nce’s picture

v1nce commented
StatusFileSize
new pager_show_all_1.patch2.89 KB

I 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.

v1nce’s picture

v1nce commented
StatusFileSize
new pager_show_all_2.patch2.89 KB

Updated patch to correctly set $pager_current.

v1nce’s picture

v1nce commented
StatusFileSize
new pager_show_all_3.patch3.33 KB

One more time. Updated patch to allow proper handling of pagination list when viewing all items.

v1nce’s picture

v1nce commented
StatusFileSize
new pager_show_all_4.patch3.23 KB

Wrong file path in above patch and removed separate feature that creeped in.

lilou’s picture

lilou commented
Status: Active » Needs work

http://drupal.org/patch/create :

Core patches should always be made from Drupal root.

jody lynn’s picture

jody lynn
she/her
Primary language English
commented
Status: Needs work » Closed (won't fix)

I 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.