When we have the pager like this:
screenshot

The current page is incorrectly identified:
https://git.drupalcode.org/project/ui_suite_bootstrap/-/blob/5.0.x/src/H...
The $variables['items']['pages'] array is like this:

[
  2 => ...
  3 => ...
  4 => ...
]

And the normalize function returns like this:

[
  0 => ...
  1 => ...
  2 => ...
]
CommentFileSizeAuthor
screenshot.png13.78 KBrpayanm

Issue fork ui_suite_bootstrap-3471652

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

rpayanm created an issue. See original summary.

rpayanm opened merge request !204

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented

I try to fix this with this patch.
Or we can keep the keys in the $page variable.

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented
Status: Active » Needs review
grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Assigned: Unassigned » grimreaper
grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Status: Needs review » Reviewed & tested by the community

Ok, so the problem happens on a full pager, when the "Number of pager links visible" is less than the number of pages.

  • grimreaper committed 806cf5dd on 5.0.x authored by rpayanm 
    Issue #3471652 by rpayanm, grimreaper: Incorrect current page detection...
grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Assigned: grimreaper » Unassigned
Status: Reviewed & tested by the community » Fixed
grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Issue tags: +Barcelona2024

Status: Fixed » Closed (fixed)

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