-
-
Notifications
You must be signed in to change notification settings - Fork 31.9k
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
[material-ui] Select components contain too many menu items with scroll, the padding right attribute is set not correctly #42304
Comments
@mj12albert Can you help to check it? Thanks. |
@Vxee I am not able to see the issue in the CodeSandbox you provided. Can you update it? |
@ZeeshanTamboli @mj12albert I updated CodeSandbox, you can try it, thanks. |
@Vxee Still not seeing the reproduction in the CodeSandbox. You just added a new image in the issue description. |
@ZeeshanTamboli Updated link, please try again. Sorry for that. |
@Vxee I'm getting "Sandbox not found" for https://codesandbox.io/p/sandbox/stupefied-morse-sh6kcj. You might need to make it public since it's currently private. |
@ZeeshanTamboli Now it is public. https://codesandbox.io/p/sandbox/stupefied-morse-sh6kcj |
It's a bug. Even without the padding-right, the padding isn't correct due to the native scrollbar overlap. I expect this to work and am surprised this issue wasn't raised before. |
Steps to reproduce
Link to live example: (required)
https://codesandbox.io/p/sandbox/stupefied-morse-sh6kcj
I override global Menu and MenuItem component style
Steps:
Current behavior
Expected behavior
Context
I have checked the Select code, here is the logic will add this logic to override the padding
Is it possible provide a prop can turn off this function?
Your environment
npx @mui/envinfo
Search keywords: Select, Menu, scroll, padding-right
The text was updated successfully, but these errors were encountered: