-
Notifications
You must be signed in to change notification settings - Fork 81
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Add Pagination component #2188
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, though would be good to add some tests for the onClick callbacks.
@@ -235,6 +235,33 @@ describe('Pagination component', () => { | |||
expect(screen.getAllByText('…')).toHaveLength(1) | |||
}) | |||
|
|||
it('can click onClickNext, onClickPrevious and onClickPagenumber', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
praise: Thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Summary
Adds a new Pagination component.
This is a port of the Pagination component that was built for the USSF Portal. I also wanted to note that I followed the instructions here and used
yarn link
to successfully test the component in the USSF Portal.Related Issues or PRs
closes: #1543
How To Test
yarn storybook
and look atComponents/Pagination/Sandbox
and change some of the controls to see how the component adjusts.yarn test:coverage
and verify that all tests are passing and that the coverage threshold is being met.Screenshots: