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.

Sign up for GitHub

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

Jump to bottom

[DataGrid] Scroll filler layout regression #14989

Open
romgrk opened this issue Oct 15, 2024 · 4 comments · May be fixed by #15000
Open

[DataGrid] Scroll filler layout regression #14989

romgrk opened this issue Oct 15, 2024 · 4 comments · May be fixed by #15000
Labels
component: data grid This is the name of the generic UI component, not the React module! regression A bug, but worse

Comments

@romgrk
Copy link
Contributor

romgrk commented Oct 15, 2024 •
edited by github-actions bot
Loading

In https://mui.com/x/react-data-grid/demo/

The scrollbar filler column is present even if no vertical scrollbar is there:

image

Search keywords:
@romgrk romgrk added component: data grid This is the name of the generic UI component, not the React module! regression A bug, but worse labels Oct 15, 2024
@k-rajat19
Copy link
Contributor

k-rajat19 commented Oct 16, 2024 •
edited
Loading

Hi @romgrk ,
I can't reproduce it on the default zoom level (which is 100%) but it appears on zoom 90% or less than that
should we still consider fixing this ?

@romgrk
Copy link
Contributor Author

romgrk commented Oct 16, 2024

Yes, it's an issue, thanks for clarifying the repro.

@k-rajat19
Copy link
Contributor

Okay can you assign me this, I'll send PR or do you have any plans working on this?

@romgrk
Copy link
Contributor Author

romgrk commented Oct 16, 2024

Sure you can take it, just open a draft PR and link to this issue (Closes #14989), that's enough to see that you're working on it.

@k-rajat19 k-rajat19 linked a pull request Oct 16, 2024 that will close this issue
Open
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component: data grid This is the name of the generic UI component, not the React module! regression A bug, but worse
Projects
MUI X Data Grid
Status: 🔖 Ready
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[docs] Fix empty space issue in PopularFeaturesDemo k-rajat19/mui-x
2 participants
@romgrk @k-rajat19