Skip to content

Show pagination across all user cards#503

Open
bipplane wants to merge 6 commits intoCATcher-org:mainfrom
bipplane:pagination-dashboard
Open

Show pagination across all user cards#503
bipplane wants to merge 6 commits intoCATcher-org:mainfrom
bipplane:pagination-dashboard

Conversation

@bipplane
Copy link
Copy Markdown
Contributor

Summary:

Fixes #470

Added and centralised the pagination range label at the bottom of each user for consistency purposes. Also reduced the coupling with the pagination arrows.

Type of change:

  • ✨ New Feature/ Enhancement
  • 🎨 Code Refactoring

Changes Made:

  • Add paginator container and range label to replace default implementations for flexible styling
  • Auto-hide navigation arrows when there is only 1 page of issues and PRs to show
  • Refactor Gradient direction syntax (fixes warning of outdated syntax)

Screenshots:

Screenshot_14

Proposed Commit Message:

Add pagination styling for card view consistency

There is no pagination range shown when the number of issues/pr cards 
is not more than the required amount of 20.

This change adds paginator components, ensuring consistent
appearance across all card view columns. It also refactors Gradient syntax 
that has been deprecated.

Checklist:

  • I have tested my changes thoroughly.
  • I have created tests for any new code files created in this PR or provided a link to a issue/PR that addresses this.
  • I have added or modified code comments to improve code readability where necessary.
  • I have updated the project's documentation as necessary.

Copy link
Copy Markdown
Contributor

@NorbertLoh NorbertLoh left a comment

Choose a reason for hiding this comment

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

The readability of pagination is definitely better than before. However, I noticed some stuff

image image

The pagination does not seem centralised.

The pagination overflows when it's 2 digits
image

@bipplane
Copy link
Copy Markdown
Contributor Author

bipplane commented Aug 7, 2025

The centralisation seems to bug out when the column is overly-wide, i'm hoping for #500 to be merged soon so that i can do further testing on this! 😅

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.

Inconsistency on showing pagination of dashboard

2 participants