MenuList doesn't recognize scrollbar size correctly.
@mnajdova is already working on this.
Since Jun 15, 2023.
- Dominant language
- JavaScript
- Stars
- 99.1k
- Forks
- 32.5k
- Avg merge
- 2d 17h
- Merged PRs (30d)
- 106
Description
Duplicates
- I have searched the existing issues
Latest version
- I have tested the latest version
Steps to reproduce 🕹
Link to live example:
https://mui.com/material-ui/react-menu/#max-height-menu
Steps:
- check MuiList's style on developer tools.
- you can find element.style is set to padding-right:0px.
Current behavior 😯
I found that MenuList is using getScrollbarSize() utils and set scrollbar size as paddingRight (on rtl mode)..
I think scrollbar size is not measured accurately. so when I set maxHeight to Menu and it render options with scrollbar, the padding-right is removed.
MenuList set element.style so I can't override that style on theme level (styleOverrides)
Expected behavior 🤔
If scrollbar size is measured to 0px, I think it should not override menu list's right padding. because default padding can be bigger than zero.
Context 🔦
No response
Your environment 🌎
npx @mui/envinfo
Don't forget to mention which browser you used.
Output from `npx @mui/envinfo` goes here.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.