Make WordPress Core

Opened 10 years ago

Last modified 5 months ago

#33405 new enhancement

Pagination - use of ellipsis / "..." not logic in some cases

Reported by: th23's profile th23 Owned by:
Milestone: Priority: lowest
Severity: trivial Version: 4.2.4
Component: Themes Keywords: has-patch close 2nd-opinion
Focuses: ui, template Cc:

Description

Hi there,

I believe there is some improvement potential in the logic of the pagination. The "..." indicator does not really serve its purpose, in case you have it only hiding one page link - see attached screenshot.

Would propose to change the function generating the pagination to only use it in case it at least covers 2 page links.

Attachments (3)

pagination.jpg (24.2 KB) - added by th23 10 years ago.
33405.diff (900 bytes) - added by luukdv 9 years ago.
33405.2.diff (1.1 KB) - added by luukdv 9 years ago.

Download all attachments as: .zip

Change History (10)

@th23
10 years ago

#1 @wonderboymusic
10 years ago

  • Keywords close added
  • Priority changed from normal to lowest
  • Severity changed from normal to trivial

#2 follow-up: @th23
10 years ago

  • Keywords close removed

Maybe its not high priority, but just closing is for me is not an option:

1) The helip requires more space than a normal number item, so on small screens showing it instead of 1 number makes it use more space in total

2) Its counter-intuitive of user expectation

Please consider fixing instead, if it's "trival"!

Thanks!

@luukdv
9 years ago

#3 @luukdv
9 years ago

Added a patch that checks if there is more than 1 page to be collapsed (with '...'), otherwise the default page link will be displayed.

#4 @luukdv
9 years ago

  • Keywords has-patch added

@luukdv
9 years ago

#5 @luukdv
9 years ago

After some additional testing with different use cases and settings, the previous solution proved to be wrong. Added a new patch.

#6 @luukdv
9 years ago

  • Keywords needs-testing added

#7 in reply to: ↑ 2 @SirLouen
5 months ago

  • Keywords close 2nd-opinion added; needs-testing removed
  • Summary changed from Pagination - use of helip / "..." not logic in some cases to Pagination - use of ellipsis / "..." not logic in some cases

Replying to th23:

Maybe its not high priority, but just closing is for me is not an option:

I'm divided between the opinion of @wonderboymusic and the potential usefulness of this report

On one side it seems something interesting and well ideated, after all that ellipsis is "bigger than a number", but on the other side it feels like a very edge case, which forces adding more code for something that is really not that prominent.

Each line of code means maintenance, and I don't really think that the fact of adding this feature could be so valuable. In fact, it has been sitting here for 10 years, and I don't see many duplicates, so it doesn't seem highly demanded.

Having rushed to do a patch was not the best idea.

To gain perspective, other CMS like XenForo, implement this one you are commenting (maybe you got this idea from a XenForo site?)

https://i.imgur.com/qbzxqh0.png
https://i.imgur.com/uhhGmp7.png

But others like SMF doesn't

https://i.imgur.com/wa1NEpA.png

Drupal has another approach, but similarly weirder (instead of just putting the 1, they put the ellipsis)

https://i.imgur.com/4ZfFaJN.png

And Joomla implements another entirely different one:

https://i.imgur.com/DkJPzJV.png

As we can see, there are no two CMS that provide the same pagination display. Also, pagination is something that could be extended, so if anyone wants a specific pagination system, you can easily write a new one.

Although, I think that more voices should comment on this topic before taking a final decision.

BTW, close tag as @wonderboymusic proposed, is just a special 2nd opinion tag to say, "I'm not completely sure if this is relevant, so I need another team member to check this and confirm, or not, if this should be closed". And this is what I'm doing now, again, but I'm going to also add 2nd-opinion, pinging some members I think might have some criteria on this topic.

cc @audrasjb @poena

Note: See TracTickets for help on using tickets.