Problem/Motivation

The mini pager has a dramatically speed improvement compared to the full pager by default.

Proposed resolution

Let's use the mini pager instead of the full pager by default, see \Drupal\views\Plugin\views\wizard\WizardPluginBase::defaultDisplayOptions
and \Drupal\views\Plugin\views\wizard\WizardPluginBase::pageDisplayOptions

Remaining tasks

Do it

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#5 default-mini-pager-2690917-5.patch1.31 KBrosinegrean
#2 default-mini-pager-2690917-2.patch1.31 KBrosinegrean

Comments

dawehner created an issue. See original summary.

rosinegrean’s picture

rosinegrean commented
Assigned: Unassigned » rosinegrean
Status: Active » Needs review
StatusFileSize
new default-mini-pager-2690917-2.patch1.31 KB

Hello,

Is something else needed ?

dawehner’s picture

dawehner
Primary language German
commented

Thank you for the really fast patch!

Good question, I would argue it depends whether this patch fails, so let's wait still for a while.

Status: Needs review » Needs work

The last submitted patch, 2: default-mini-pager-2690917-2.patch, failed testing.

rosinegrean’s picture

rosinegrean commented
Status: Needs work » Needs review
StatusFileSize
new default-mini-pager-2690917-5.patch1.31 KB

@dawehner forgot to rebase my local repo.

Here it goes again.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Nice, its also green now!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed 7c5c0d6 and pushed to 8.1.x and 8.2.x. Thanks!

  • alexpott committed 1b3b670 on 8.2.x 
    Issue #2690917 by prics, dawehner: Use mini pager by default for new...

  • alexpott committed 7c5c0d6 on 8.1.x 
    Issue #2690917 by prics, dawehner: Use mini pager by default for new...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.