Currently the views mini pager is shown even if there is only one page of items. IMHO the pager should not be shown if there is only one page of items. This is also how the way how Drupal Core handles this (views-mini-pager.html.twig).

I suppose the problem arises because Radix is using the same twig for full pager and mini pager. Mini pager is handling it different from the full pager. Full pager has an empty items array if there is only one page, while mini pager fills the items array even if there is only one page.

I propose a patch to get this in line with the Drupal core behavior.

Issue fork radix-3084620

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:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

stefan.korn created an issue. See original summary.

stefan.korn’s picture

Assigned: stefan.korn » Unassigned
Status: Active » Needs review
FileSize
495 bytes
shadcn’s picture

Status: Needs review » Reviewed & tested by the community

Tested. Looks good. RTBC. Thanks Stefan for all the patches. Really appreciate this.

stefan.korn’s picture

@arshadcn: Thanks for thanks :-) - I really love Radix

  • arshadcn committed e62e316 on 8.x-4.x authored by stefan.korn
    Issue #3084620 by stefan.korn: Views Mini pager should not show if there...
shadcn’s picture

Status: Reviewed & tested by the community » Fixed

Committed. 👍

Status: Fixed » Closed (fixed)

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

liquidcms made their first commit to this issue’s fork.