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
[Pagination] Fix display when boundaryCount={0} #21446
Conversation
Details of bundle changes.Comparing: 7a1262d...9bf3559 Details of page changes
|
…ryCount is zero. Issue mui#21418.
argos test is showing differences on demo https://material-ui.com/pt/components/lists/ |
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.
Thanks for working on this.
Could you include tests in packages/material-ui-lab/src/Pagination/Pagination.test.js
that were failing before this change?
Our test setup is a bit flaky when it comes to image loading. The demo is loading images so screenshots might not be taken at the same time. We can manually approve these "fails" |
@guimacrf It's a great first pull request on Material-UI 👌🏻. Thank you for working on it! |
Thank you! Glad for helping! |
Edit @eps1lon:
Closes #21418