search

mui / mui-x

MUI X: Build complex and data-rich applications using a growing list of advanced React components, like the Data Grid, Date and Time Pickers, Charts, and more!
https://mui.com/x/
4.53k stars 1.32k forks source link

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

Closed djairhogeuens closed 20 hours ago

djairhogeuens commented 1 month ago

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

romgrk commented 1 month ago

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

KenanYusuf commented 1 month ago

@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.

kalyan90 commented 1 month ago

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

romgrk commented 1 month ago

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 commented 1 month ago

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 https://github.com/mui/mui-x/pull/14864. Thank you

KenanYusuf commented 20 hours ago

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

github-actions[bot] commented 20 hours ago

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.