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

[data grid] Excessive whitespace when scrolling to right #14816

Closed
djairhogeuens opened this issue Oct 3, 2024 · 7 comments
Closed

[data grid] Excessive whitespace when scrolling to right #14816

djairhogeuens opened this issue Oct 3, 2024 · 7 comments
Labels
bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module!

Comments

@djairhogeuens
Copy link

djairhogeuens commented Oct 3, 2024

Steps to reproduce

Link to live example: https://mui.com/material-ui/getting-started/templates/dashboard/

Steps:
Scroll to the right in the data grid

Current behavior

For some reason, there is an excessive white space on the right of the data grid within it's container. You can see it when you scroll to the right in the data grid.

Expected behavior

No possibility to scroll right if all content is displayed in the container. Wen scrolling to the right is allowed due to not all content being displayed in the container, the scroll should stop at the end of the table without any excessive whitespace.

Context

No response

Your environment

Chrome Version 129.0.6668.60 (Official Build) (64-bit)

Search keywords: data grid, scroll

@djairhogeuens djairhogeuens added bug 🐛 Something doesn't work status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Oct 3, 2024
@romgrk
Copy link
Contributor

romgrk commented Oct 3, 2024

@KenanYusuf I think you worked on something related to this recently, could you handle this one?

@KenanYusuf
Copy link
Member

@romgrk I can take a look at this in a couple of weeks (occupied with mobile view work until then). If anyone wants to take a look at this before then, please feel free.

@oliviertassinari oliviertassinari added component: data grid This is the name of the generic UI component, not the React module! and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Oct 3, 2024
@oliviertassinari oliviertassinari changed the title Excessive whitespace when scrolling to right in data grid [data grid] Excessive whitespace when scrolling to right Oct 3, 2024
@kalyan90
Copy link
Contributor

kalyan90 commented Oct 7, 2024

@romgrk @michelengelen I can try check on this issue, would I need to sign CLA?

@romgrk
Copy link
Contributor

romgrk commented Oct 7, 2024

Not unless you touch a file inside the *-(pro|premium) packages, but layout bugs should be in the core (aka community) packages. Feel free to look into it. Open a draft PR if you want to take it, otherwise I'll look into it myself.

IME, that issue is caused by dangling/floating elements that poke outside of their container's width, usually in the headers container.

@kalyan90
Copy link
Contributor

kalyan90 commented Oct 7, 2024

Not unless you touch a file inside the *-(pro|premium) packages, but layout bugs should be in the core (aka community) packages. Feel free to look into it. Open a draft PR if you want to take it, otherwise I'll look into it myself.

IME, that issue is caused by dangling/floating elements that poke outside of their container's width, usually in the headers container.

@romgrk added a PR for this issue #14864. Thank you

@KenanYusuf
Copy link
Member

Looks like this was solved with #14864 - thanks @kalyan90.

Copy link

This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue.
Now, if you have additional information related to this issue or things that could help future readers, feel free to leave a comment.

Note

@djairhogeuens How did we do? Your experience with our support team matters to us. If you have a moment, please share your thoughts in this short Support Satisfaction survey.

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

No branches or pull requests

5 participants