-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
[DataGrid] Fix print export failure when hideFooter
option is set
#13034
Conversation
5330d84
to
6456fda
Compare
@michelengelen, mind taking a look? |
packages/x-data-grid/src/hooks/features/export/useGridPrintExport.tsx
Outdated
Show resolved
Hide resolved
@cherniavskii or @MBilalShafi gentle ping on this one! |
6456fda
to
d241433
Compare
packages/x-data-grid/src/hooks/features/export/useGridPrintExport.tsx
Outdated
Show resolved
Hide resolved
hideFooter
option is set
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. Thank you @tarunrajput for working on it.
Deploy preview: https://deploy-preview-13034--material-ui-x.netlify.app/ |
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.
sry for the block! 👍🏻
…ui#13034) Co-authored-by: Bilal Shafi <bilalshafidev@gmail.com>
…ui#13034) Co-authored-by: Bilal Shafi <bilalshafidev@gmail.com>
Fixes: #13015
When
hideFooter
option is true, grid footer is not rendered anymore. This PR adds a check to skip styling the grid footer element in that case.