Problem/Motivation

The feed display in default taxonomy term view uses a full pager. Changing it to some or mini would save one count query.

Proposed resolution

Remove the pager and change the number of items from 15 back to 10.

Remaining tasks

User interface changes

API changes

Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
Issue category Task because nothing is broken. This is about removing one extra count query.
Issue priority Not critical.
Prioritized changes The main goal of this issue is performance.

Comments

dawehner’s picture

This is the exact the same behaviour as in d7 ... which used to be 10 items, maybe we want to change this here?

olli’s picture

StatusFileSize
new1.13 KB
new534 bytes

True.

jibran’s picture

Do you think we should add test for this? Other then that it is RTBC.

jibran’s picture

This is a task and I think we can live without tests here. But it needs beta evaluation for RTBC.

olli’s picture

Related issues: +#2469157: Change default views to use the mini pager
StatusFileSize
new1.14 KB

reroll.

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Looks still fine for me.

not sure whether its worth adding a test which tests something we have already test coverage for.

alexpott’s picture

Status: Reviewed & tested by the community » Needs work

I don;t think a test is needed. This is a config only change with 0 disruption. But as per #4 we do need a beta eval.

olli’s picture

Issue summary: View changes
Status: Needs work » Reviewed & tested by the community
Issue tags: -Needs issue summary update

Added beta phase evaluation.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.0.x. Thanks!

  • webchick committed b8651a6 on 8.0.x
    Issue #2375773 by olli, dawehner: Remove pager from default taxonomy...

Status: Fixed » Closed (fixed)

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