The original pager implementation was scrapped from the Seven theme issue (#484860: Initial D7UX admin theme) because it requires another core patch applied, namely #514914: Re-add removed feature of $limit "passing" for themers, and that dependency was uncomfortable for that issue as far as I've understood. Now, we should fix this issue, since the current pager looks less informative, so here is a follow up with the pager.

Pager looks in Seven before this patch:

After this patch:

Still requires #514914: Re-add removed feature of $limit "passing" for themers which should be committed first or this patch is totally broken.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rszrama’s picture

Status: Needs review » Reviewed & tested by the community

Works for me. Not sure if there's supposed to be any discussion over this new pager's functionality, but if this is the design then why not go for it? Sorry, still a Drupal issue tracker newb, but is it ok to RTBC like this?

rszrama’s picture

Status: Reviewed & tested by the community » Needs work

I may have spoke too soon. I noticed a pager saying "1 - 0 of 19" on the taxonomy page when I was adding terms to a vocabulary. Without this patch, the normal pager isn't displayed at all. See image:

Gábor Hojtsy’s picture

Interesting, since the issue being committed at #514914: Re-add removed feature of $limit "passing" for themers should have made this work. Hum. Are you sure you work with an up to date Drupal 7 checkout?

Gábor Hojtsy’s picture

Status: Needs work » Needs review
FileSize
27.33 KB
3.33 KB

I've reapplied this patch to my testing environment and it still works as advertised above. Now that it does not have any dependencies. It would be great to verify this independently, but I still see this working. In the watchdog reports table:

Patch rerolled because it had some fuzz applying.

Status: Needs review » Needs work

The last submitted patch failed testing.

rszrama’s picture

I'll try it again after I restore my D7 dev site... something broke it last night. : P

fwiw, I had just checked out D7 yesterday from HEAD when I saw that problem on a taxonomy term list page.

Gábor Hojtsy’s picture

Status: Needs work » Needs review
FileSize
42.47 KB
3.33 KB

Here is a fresh reroll of the patch. Tested again, and it still works for me. Anyone else to verify?

yoroy’s picture

Havent' looked at it in IE, but FF and Safari looking dandy.

mr.baileys’s picture

FileSize
163.54 KB

Tested in IE 7 (some other items on the page are screwy, but both before and after patch), FF 3.5 and Safari 3 and the pager looks (and works) great, definitely an improvement. I was unable to reproduce the problem described in #2 and the pager worked consistently and had the correct numbers.

My only concern so far is

+.short-pager {
+  float: right;
+  margin-top: 2em;
+}

I've always been told that floats should have an explicit width (although it seems the sentence "a floated box must have an explicit width" has recently been removed from the CSS2.1 specs). What is the reasoning behind using float:right instead of text-align:right? Not sure the positioning of the element following the pager is ideal when using floats (see screenshot).

yoroy’s picture

FileSize
3.34 KB

changed the float to a text-align

mattyoung’s picture

.

xmacinfo’s picture

If we want to use this new pager, which looks nice we need to make it

- easier to jump forward and backward,
- to jump to a specific page and also
- to tell the exact number of remaining items instead of showing the paging value.

We must keep First and Last. Example:

«First ‹Previous 50 Showing 100 – 150 of 250 Next 50› Last»

In this example, the 100 in bold should be clickable to show a simple form field, aka edit in place where the user would type a number to jump to the record.

Paging through a long list can be easy or painful. Let's not make this patch painful. :-)

Also, in this example:

«First ‹Previous 50 Showing 100 – 150 of 156 Next 6› Last»

... the Next part should be intelligent enough to display 6 (156 - 150). Using the 50 items on a page value is wrong here. Make it intelligent.

Status: Needs review » Needs work

The last submitted patch failed testing.

mattyoung’s picture

@xmacinfo:

>the 100 in bold should be clickable to show a simple form field, aka edit in place where the user would type a number to jump to the record

You mean like this: Jump pager?

xmacinfo’s picture

@mattyoung:

Yes. This Minimax page module expose the current page number in a field for easy jump. But if we want a nice styling and look à la Seven, we could hide the field and expose it only when clicking on the current page number.

For example, clicking on the current page (or range):

«First ‹Previous 50 Showing 100 – 150 of 156 Next 6› Last»

would become

«First ‹Previous 50 [ 100 ] of 156 Next 6› Last»

where [ 100 ] would be the exposed field that would serve as jump. And in this example value permitted would be from 1 to 156 since we know that the max known is 156.

rickvug’s picture

Status: Needs work » Needs review

#10: pagers.patch queued for re-testing.

Sivaji_Ganesh_Jojodae’s picture

FileSize
4.15 KB

+1 for this pager it looks much better. The above patch fails to display link to first and last page which is required for easy navigation and it no longer works because the theme_pager_link() function signature has been changed as a part of #572618: All theme functions should take a single argument to make preprocess sane and meaningful. Attached patch adds link to first and last page and changes to match signature.

xmacinfo’s picture

Status: Needs review » Needs work

Please provide a screen capture.

Also, please add a jump box in the middle of the pager. Without a jump box, this sort pager, although more beautiful, is not usable. See my comment in #15.

Sivaji_Ganesh_Jojodae’s picture

I have attached screenshots with this comment, will re-roll a patch with jump box similar to 538788_reroll.png.

BTW : How do i add screenshot inline like #7 ?

xmacinfo’s picture

Looks nice! Adding the jump box will make it ready to rock.

A small note, in the intermediate version of the screen capture, the

... showing 51 - 100 of 207 ...

uses a range (51 to 100), which is fine in itselft. However, when combined with a jumpbox, it becomes tricky, since we might not want to ask the user to input the value of both start and end range. We should instead ask only for the start of the range:

... showing [50] - 100 of 207 ...

What do you think?

Sivaji_Ganesh_Jojodae’s picture

Status: Needs work » Needs review
FileSize
5.64 KB
7.02 KB

Attached patch provides a way for quick navigation as mentioned in #18.

@xmacinfo, I am using list select instead of inputbox see the attached snapshot and patch. I know the patch needs work, i like to confirm that i am on right track here.

Status: Needs review » Needs work

The last submitted patch, 538788_seven_pager_21.patch, failed testing.

xmacinfo’s picture

Version: 8.x-dev » 7.x-dev

Indeed, select is the way to go for ranges. Much more clean. But I would include the whole range in the select input items:

« First « Prev 50 [400 - 450] of 1001 Next 50 » Last »

[400 - 450] => the displayed range in an item in the select list...

I cannot review the rest of your patch or the TO DO! for new. :-)

Bojhan’s picture

Version: 7.x-dev » 8.x-dev

I am going to push this feature to Drupal 8. Looking at the progress so far and considering that we are deep into UI freeze.

This feature has no clear usability win, although Mark Boulton did indeed design it this way and we are striving to implent it as such, there is no sign that we are actually improving anything. Sure with the latest patch I can navigate to page 77, but who knows whats on page 77?

Our current pager, although not the best looking one out there - has no known usability flaws, nor is its inability to jump to be found a huge problem.

Jeff Burnz’s picture

Version: 7.x-dev » 8.x-dev

sub

droplet’s picture

@Bojhan,

users know what's on Page 77.

eg. I filtered a list of articles and navigate to page 77. I would know my content is on page 77 now. when i go to other computer or next time to edit the article I will jump to page 77 directly.

Gaelan’s picture

Status: Needs work » Needs review

#21: 538788_seven_pager_21.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 538788_seven_pager_21.patch, failed testing.

JohnAlbin’s picture

Issue tags: +mobile

Marking #1488866: Redesign pagers for administrative lists. as a dupe.

An improved pager would make things easier for mobile users as well, as long as the touch targets were large enough.

mondrake’s picture

Hi all, you may want to take a look at the Pagerer module I recently contributed. http://drupal.org/project/pagerer

mgifford’s picture

This is going to have to change when Views comes into D8 Core, right? Otherwise it's just a matter of getting someone to look at migrating the patch from #21 to the new /core structure.

@Bojhan, I'm assuming that this is both a usability improvement as well as just a matter of following the original design, right?

How do we get this moving so that Seven looks better than it did in D7?

LewisNyman’s picture

Status: Needs work » Closed (duplicate)

I'm going to consider this an duplicate of #2016867: Pager Style Update. If anyone has any objections to the pager design found in the new Seven style guide we can take it up there.

xmacinfo’s picture

Status: Closed (duplicate) » Needs work

The issue at #2016867: Pager Style Update is a complete regression that does not take into account any discussion taken here. Please do not sidetrack.

LewisNyman’s picture

Ok, but I do think it's counter productive to have the two discussions completely separate. We can't commit both.

LewisNyman’s picture

Ok, we've managed to figure out a good separation between issues. See my comment here:
https://drupal.org/node/2016867#comment-7572783

Keep up the good work!

Bojhan’s picture

So what is this issue still about?

Bojhan’s picture

Version: 8.0.x-dev » 8.1.x-dev
Category: Task » Feature request
Issue summary: View changes

Discussed this with Lewis

This is really a new feature and it would be nice, but not really needed for release.

Version: 8.1.x-dev » 8.2.x-dev

Drupal 8.1.0-beta1 was released on March 2, 2016, which means new developments and disruptive changes should now be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.2.x-dev » 8.3.x-dev

Drupal 8.2.0-beta1 was released on August 3, 2016, which means new developments and disruptive changes should now be targeted against the 8.3.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.3.x-dev » 8.4.x-dev

Drupal 8.3.0-alpha1 will be released the week of January 30, 2017, which means new developments and disruptive changes should now be targeted against the 8.4.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Gábor Hojtsy’s picture

Version: 8.4.x-dev » 8.5.x-dev

Drupal 8.4.0-alpha1 will be released the week of July 31, 2017, which means new developments and disruptive changes should now be targeted against the 8.5.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.5.x-dev » 8.6.x-dev

Drupal 8.5.0-alpha1 will be released the week of January 17, 2018, which means new developments and disruptive changes should now be targeted against the 8.6.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.6.x-dev » 8.7.x-dev

Drupal 8.6.0-alpha1 will be released the week of July 16, 2018, which means new developments and disruptive changes should now be targeted against the 8.7.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.7.x-dev » 8.8.x-dev

Drupal 8.7.0-alpha1 will be released the week of March 11, 2019, which means new developments and disruptive changes should now be targeted against the 8.8.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.8.x-dev » 8.9.x-dev

Drupal 8.8.0-alpha1 will be released the week of October 14th, 2019, which means new developments and disruptive changes should now be targeted against the 8.9.x-dev branch. (Any changes to 8.9.x will also be committed to 9.0.x in preparation for Drupal 9’s release, but some changes like significant feature additions will be deferred to 9.1.x.). For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

Version: 8.9.x-dev » 9.1.x-dev

Drupal 8.9.0-beta1 was released on March 20, 2020. 8.9.x is the final, long-term support (LTS) minor release of Drupal 8, which means new developments and disruptive changes should now be targeted against the 9.1.x-dev branch. For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

Version: 9.1.x-dev » 9.2.x-dev

Drupal 9.1.0-alpha1 will be released the week of October 19, 2020, which means new developments and disruptive changes should now be targeted for the 9.2.x-dev branch. For more information see the Drupal 9 minor version schedule and the Allowed changes during the Drupal 9 release cycle.

Version: 9.2.x-dev » 9.3.x-dev

Drupal 9.2.0-alpha1 will be released the week of May 3, 2021, which means new developments and disruptive changes should now be targeted for the 9.3.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.3.x-dev » 9.4.x-dev

Drupal 9.3.0-rc1 was released on November 26, 2021, which means new developments and disruptive changes should now be targeted for the 9.4.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.4.x-dev » 9.5.x-dev

Drupal 9.4.0-alpha1 was released on May 6, 2022, which means new developments and disruptive changes should now be targeted for the 9.5.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.5.x-dev » 10.1.x-dev

Drupal 9.5.0-beta2 and Drupal 10.0.0-beta2 were released on September 29, 2022, which means new developments and disruptive changes should now be targeted for the 10.1.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

quietone’s picture

Project: Drupal core » Seven
Version: 10.1.x-dev » 1.0.0-alpha1
Component: Seven theme » Code

I asked for help in #frontend to move Seven and Classy issues from the core component to the correct place, since Seven and Classy have been removed from core. markconroy responded saying this issue should move to the Seven project in contrib. They were not able to do that because the meta data was not visible (don't know why). So, I am moving this issue. markconroy already has credit on the Seven triage meta issue so there is no need to add credit. #3332483: [Meta] Seven issue triage