Feature #25490

pagination is fairly simple and not usable in bigger projects

Added by Georg Ringer almost 11 years ago. Updated almost 11 years ago.

Status:
Resolved
Priority:
Should have
Category:
Widgets
Target version:
-
Start date:
2011-03-31
Due date:
% Done:

100%

Estimated time:
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.


Files

fluid_pagination.patch (8.34 KB) fluid_pagination.patch Georg Ringer, 2011-03-31 22:39
#1

Updated by Georg Ringer almost 11 years ago

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

#2

Updated by Mr. Hudson almost 11 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 almost 11 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 almost 11 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 almost 11 years ago

  • Assignee 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 almost 11 years ago

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

#7

Updated by Mr. Hudson almost 11 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 almost 11 years ago

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

Also available in: Atom PDF