Feature #25490

pagination is fairly simple and not usable in bigger projects

Added by Georg Ringer over 4 years ago. Updated over 4 years ago.

Status:Resolved Start date:2011-03-31
Priority:Should have Due date:
Assigned To:Bastian Waidelich % Done:

100%

Category:Widgets
Target version:-
Has patch:

Description

The pagination widget is fairly simple and not usable in bigger projects with more than 50 records. The reason is that it just renders all links which are needed. So if there are 1000 records and 10 per page, this results in 100 links on the page.

Therefore it is needed to be able to skip the rendering of links at some point and implement something like:
< | 1 | 2 | 3 | [...] | 190 | 191 | 192.

Additionally the pagination uses some hardcoded labels which could be fixed with the same patch.

fluid_pagination.patch Magnifier (8.3 kB) Georg Ringer, 2011-03-31 22:39

Associated revisions

Revision e77ec815
Added by Tobias Liebig over 4 years ago

[FEATURE] improve fluid paginate widget

  • forward port changes from extbase
  • add configuration for maximum number of pages to display

Thanks to Georg Ringer for the initial patch.
Thanks to HDNET GmbH for sponsoring 120 beers
during the bug auction.

Change-Id: Ic74cdda291b987761c9715b5685d1b32b7e8f063
Resolves: #25490

History

#1 Updated by Georg Ringer over 4 years ago

can someone do the git stuff, don't have the proper tools here at t3board, sry guys

#2 Updated by Mr. Hudson over 4 years ago

Patch set 1 of change Ic74cdda291b987761c9715b5685d1b32b7e8f063 has been pushed to the review server.
It is available at http://review.typo3.org/1388

#3 Updated by Mr. Hudson over 4 years ago

Patch set 2 of change Ic74cdda291b987761c9715b5685d1b32b7e8f063 has been pushed to the review server.
It is available at http://review.typo3.org/1388

#4 Updated by Mr. Hudson over 4 years ago

Patch set 3 of change Ic74cdda291b987761c9715b5685d1b32b7e8f063 has been pushed to the review server.
It is available at http://review.typo3.org/1388

#5 Updated by Tobias Liebig over 4 years ago

  • Assigned To set to Bastian Waidelich
  • % Done changed from 50 to 80

thanks for your initial patch.

First I did some forward porting of the paginate widget from extbase/fluid to FLOW3/Fluid to make your patch apply in Fluid(v5). Then i did some renaming and minor refactoring.

The original code is from EXT:pagebrowse by Dmitry, right? We should mention that somewhere.

Once the patch is merged, it should be backported/synced to extbase/fluid, too.

Assigned to Bastian for review.

#6 Updated by Georg Ringer over 4 years ago

yes original code is from pagebrowse. thanks for your work!

#7 Updated by Mr. Hudson over 4 years ago

Patch set 4 of change Ic74cdda291b987761c9715b5685d1b32b7e8f063 has been pushed to the review server.
It is available at http://review.typo3.org/1388

#8 Updated by Tobias Liebig over 4 years ago

  • Status changed from New to Resolved
  • % Done changed from 80 to 100

Also available in: Atom PDF