Use light (lite) pager by default [#1909474]

#1901290: Replace the mini pager with a lite pager (which does not run a count query) adds a 'lite' pager to Views, this only has previous/next links and hence doesn't need to do a COUNT() query. This can in many cases halve the time Views spends querying the db (by halving the queries), so we should just use it by default. It's also a lot tidier.

Lots of sites now use infinite scroll or lite pagers rather than numbers/previous/next/last so this isn't going to be jarring I don't think.

Comment	File	Size	Author
#2	drupal-1909474-2.patch	2.82 KB	dawehner
#2

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

dawehner

German

CreditAttribution: dawehner commented 6 February 2013 at 22:26

So it should not only be the default to use, but it should also be replaced in all the test views if possible.

There is not only the performance advantage but it actually works more independent of sql, so we might should reconsider the SqlPluginBase naming, but yeah that's out of scope here.

Comment #2

dawehner

German

CreditAttribution: dawehner commented 18 February 2013 at 19:30

Status:

Active

» Needs review

File	Size
drupal-1909474-2.patch	2.82 KB

Left out the frontpage view, as I didn't wanted to break the other patch.

Comment #3

damiankloip CreditAttribution: damiankloip commented 24 February 2013 at 11:46

The patch looks fine, but as it's a simple patch that's easy to re roll, maybe we should postpone this on #1806334: Replace the node listing at /node with a view as that is rtbc now?

Comment #4

dawehner

German

CreditAttribution: dawehner commented 24 February 2013 at 23:07

If you look at the patch: This one actively not uses the frontpage view, we can do this two patches independent from each other.

Comment #5

damiankloip CreditAttribution: damiankloip commented 25 February 2013 at 07:58

Ah ok, I see, so the /node issue is not going to use the light pager?

Comment #6

dawehner

German

CreditAttribution: dawehner commented 25 February 2013 at 11:42

Well, we can, but as this is a feature change visible on the actual page we might should talk about it (remember, all other default views are just examples that are disabled out of the box).