Skip to content

removing custom pager template that uses ilw-pagination#1312

Open
nhjohnstone wants to merge 1 commit into
5.xfrom
nhjohnstone/issue-1311-remove-custom-pager-template-and-style-to-look-like-the-toolkit-pager
Open

removing custom pager template that uses ilw-pagination#1312
nhjohnstone wants to merge 1 commit into
5.xfrom
nhjohnstone/issue-1311-remove-custom-pager-template-and-style-to-look-like-the-toolkit-pager

Conversation

@nhjohnstone

Copy link
Copy Markdown
Contributor

📝 Description

Removed custom pager template and styled the stable9 pager instead

Fixes #1311


✅ Peer Review Checklist

Reviewers: Please verify the following before approving.

  • The pull request links to the issue it is addressing in the comment and in the "Development" section of the PR
  • There is a short summary that describes how the fix is implemented
  • Verifying work: Verify that the changes made are addressing the linked issue
  • Accessibility: Changes that might impact accessibility are reviewed to work with keyboard navigation and screen readers
  • Responsive: Layout is tested on desktop, tablet, and mobile screens
  • Config changes: Any configuration updates are tested and verified using the Distribution Update import
  • Security: Output is sanitized (using t(), Twig auto-escaping, etc.). No secrects included (API keys, etc.)
  • Cleanliness: All debug code (ksm(), dpm(), console.log) has been removed.
  • Comments: Complex logic is explained inline.
  • Documentation: Any relevant documentation has been updated (this can be a separate issue if needed)

📸 Screenshots / Video (Optional)

If this is a UI change, please attach a screenshot or a quick screen recording of the change in action.

@nhjohnstone nhjohnstone requested a review from trubach June 23, 2026 14:38

@trubach trubach left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Custom pager looks good! Ready to merge!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove custom pager template and style to look like the toolkit pager

2 participants