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
| 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. |
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 2375773-5.patch | 1.14 KB | olli |
Comments
Comment #1
dawehnerThis is the exact the same behaviour as in d7 ... which used to be 10 items, maybe we want to change this here?
Comment #2
olli commentedTrue.
Comment #3
jibranDo you think we should add test for this? Other then that it is RTBC.
Comment #4
jibranThis is a task and I think we can live without tests here. But it needs beta evaluation for RTBC.
Comment #5
olli commentedreroll.
Comment #6
dawehnerLooks still fine for me.
not sure whether its worth adding a test which tests something we have already test coverage for.
Comment #7
alexpottI 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.
Comment #8
olli commentedAdded beta phase evaluation.
Comment #9
webchickCommitted and pushed to 8.0.x. Thanks!