Skip to content

Added pagination#109

Merged
jarenadams21 merged 1 commit intomainfrom
jane-pagination
Apr 9, 2025
Merged

Added pagination#109
jarenadams21 merged 1 commit intomainfrom
jane-pagination

Conversation

@janekamata
Copy link
Contributor

ℹ️ Issue

Closes #102

📝 Description

  • Moved pagination back into GrantList component
  • Fixed issues with pagination and kept use of Chakra
  • Matched formatting to Figma

✔️ Verification

  • Tested pagination by clicking on buttons
  • Tested by sorting and making sure the pagination works on the sorted list
  • Tested ellipsis by changing number of items showing to 1

image

🏕️ (Optional) Future Work / Notes

Code could be cleaned up / refactored but pagination only being used here...

@janekamata janekamata self-assigned this Apr 2, 2025
@janekamata janekamata requested a review from jarenadams21 April 2, 2025 03:49
Copy link
Collaborator

@jarenadams21 jarenadams21 left a comment

Choose a reason for hiding this comment

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

verified on my own local, thanks!

@jarenadams21 jarenadams21 merged commit 0e484d2 into main Apr 9, 2025
1 check passed
@prooflesben prooflesben deleted the jane-pagination branch October 4, 2025 18:45
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.

[MARCH-5] Fix Pagination

2 participants