[TablePagination] Add showFirstButton and showLastButton support#20750
[TablePagination] Add showFirstButton and showLastButton support#20750oliviertassinari merged 7 commits intomui:nextfrom
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
|
eps1lon
left a comment
There was a problem hiding this comment.
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
eps1lon
left a comment
There was a problem hiding this comment.
It's not clear to me why the title was added. Especially since there are no tests for it.
eps1lon
left a comment
There was a problem hiding this comment.
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
getItemAriaLabelprop.fixes #20725
Look
Testing video
https://recordit.co/1zDeLDC3zn