Skip to content

[DashboardLayout] Prevent horizontal scrollbars by adding tolerance#4861

Merged
apedroferreira merged 2 commits intomui:masterfrom
bharatkashyap:fix/mini-scroll-x
Apr 18, 2025
Merged

[DashboardLayout] Prevent horizontal scrollbars by adding tolerance#4861
apedroferreira merged 2 commits intomui:masterfrom
bharatkashyap:fix/mini-scroll-x

Conversation

@bharatkashyap
Copy link
Collaborator

@bharatkashyap bharatkashyap added the design: ui Visual design. label Apr 18, 2025
@mui-bot
Copy link

mui-bot commented Apr 18, 2025

Netlify deploy preview

https://deploy-preview-4861--mui-toolpad-docs.netlify.app/

Generated by 🚫 dangerJS against fbfa25c

Copy link
Collaborator

@apedroferreira apedroferreira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this works, thanks!

Copy link
Collaborator

@apedroferreira apedroferreira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait I tried it in the playground and it still might not be working?
Looking into this, it works in the demos...

Edit: Probably feel free to add overflowX: 'hidden' in the container if it does the job, looks like it does.
And we can revert the -2 we added here.
I guess this overflow is caused by the vertical scrollbar itself so this should be fine.

Screenshot 2025-04-18 at 12 11 48

@apedroferreira apedroferreira merged commit ff15f61 into mui:master Apr 18, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

design: ui Visual design.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants