-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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
[TablePagination] Add showFirstButton and showLastButton support #20750
Conversation
packages/material-ui/src/TablePagination/TablePaginationActions.js
Outdated
Show resolved
Hide resolved
@ShahAnuj2610 Thanks for looking into it, we will get it land into v5. We have freezed new features on v4. |
packages/material-ui/src/TablePagination/TablePaginationActions.js
Outdated
Show resolved
Hide resolved
@material-ui/core: parsed: +0.25% , gzip: +0.38% Details of bundle changes.Comparing: a8b54b7...aae2c58 Details of page changes
|
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.
Could you add some tests in TablePagination.test.js
? Especially handleLastPageButtonClick
looks prone to off-by-one errors.
@eps1lon have added test cases for |
2feb2d1
to
33dbdfc
Compare
ebe242a
to
e73f6a1
Compare
6c22cb4
to
707c252
Compare
a08ea7b
to
d018ce9
Compare
9317d05
to
ca3e6bf
Compare
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.
It's not clear to me why the title
was added. Especially since there are no tests for it.
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.
I guess the title usage is more widespread. We probably want to get rid of this in the future and replace with Tooltips
Breaking change
getItemAriaLabel
prop.fixes #20725
Look
Testing video
https://recordit.co/1zDeLDC3zn