Closed (works as designed)
Project:
Views (for Drupal 7)
Version:
8.x-3.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
12 Jan 2011 at 13:07 UTC
Updated:
2 Nov 2019 at 20:27 UTC
Jump to comment: Most recent
Comments
Comment #1
dawehnerThis is how drupal works. See drupal.org as example.
Comment #2
tsvenson commentedI'm aware of that this is how Drupal works. Just hoped it was something that wasn't to complicated to add as for humans it would make more sense when looking at the URL's.
Comment #3
iamsquared commentedDereine, many talk about making Drupal more user friendly. I respect that is works as intended. But Tsvenson has a point that humans expect the first page of a result set to be page "1" not the computer-like "0". Maybe it does "work as designed" for computers but it would be nice to see respect for making Drupal more user friendly too. Having a simple checkbox & field like, "[x] start numbering results from __" seems like what Tsvenson was asking for, and it would have the added benefit of being able to 'sync' views to something like 'nodequeue' which starts at "1", too.
Comment #4
merlinofchaos commentedViews' pager works the same way that Drupal default pagers.
Someone in contrib is free to make a pager plugin that works the way you want it to work, but this won't be in core Views. The vast majority of administrators do not care about this, so the option would simply be confusing. For those that do, that's why we have contrib and a pluggable architecture.
Comment #5
xpersonas commentedI agree that this would be a nice feature. I get that it won't be added to views and that it's how Drupal works. But it would be nice. So there are two admins and a few of my users that would like the feature. That still leaves us out of the vast majority, but figured I would second it anyway.
Comment #6
Dharmendra.s commentedi don't know how to create patch , but i have modified pager.inc file and it works fine for me..
find includes/pager.inc
function theme_pager($tags = array(), $limit = 10, $element = 0,
$parameters = array(), $quantity = 5) {
122 - $pager_current = $pager_page_array[$element] +1;
+ $pager_current = $pager_page_array[$element];
and
129 - $pager_max = $pager_total[$element];
+ $pager_max = $pager_total[$element] - 1;
Comment #7
yuriy.sychenko commentedI used your patch works for me, but there is a catch after using it, I was gone in the first page of the previous and not highlighted one active link in the pager, and when I go to a link in the one pager that I get to get on page 2.
Comment #8
tsvenson commentedI'm re-opening this one for 8.x and tagging it with VDC.
Reason is the same as given in #2 and #3. We should aim for improving the UX for human users/visitors, I think that is a good thing.
Comment #9
damiankloip commentedUnless this directive changes in core, this will not change in views. Even more so with VDC..
If you feel strongly about this, I would recommend maybe opening a core issue instead.
Comment #10
tsvenson commented@damiankloip, one step ahead of you #1818040: Pager should start counting from 1, not 0
Comment #11
mondrakeI am trying to implement this in the contrib Pagerer module, see #3091945: Allow Pagerer to override the pager querystring part in URLs and count pages from 1, not 0. Reviews, tests, suggestions would be appreciated there.