Skip to content
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 footer count not showing on small screens #899

Merged
merged 5 commits into from
Jan 23, 2021
Merged

[DataGrid] Fix footer count not showing on small screens #899

merged 5 commits into from
Jan 23, 2021

Conversation

mnajdova
Copy link
Member

Closes #847

I've tried the silution proposed in #847 (comment) but there was a problem (when there is no select for rows per page, the caption which we want to show is immediately after the $spacer and it was hidden).

The only viable difference I could find between the two captions, was that the one which is used as label have id, so I used that information. Honestly, it would have been much easier, if the TablePagination had different classes for these two elements.

Not sure if this is the best fix, but I am open to other ideas. I was trying to come up with some sibling selector, but I couldn't do it, as the same class is used on both places.

Not sure what's the best way for adding test for this, so please let me know.

@mnajdova mnajdova added bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module! labels Jan 22, 2021
@DanailH DanailH requested a review from dtassone January 22, 2021 14:36
@oliviertassinari oliviertassinari changed the title [DataGrid] Footer count not shown on small screen [DataGrid] Fix footer count not showing on small screens Jan 22, 2021
@mnajdova mnajdova merged commit cdb8a96 into mui:master Jan 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DataGrid] Footer count not showing in "official" demo itself
4 participants