mui / mui/material-ui

MenuList doesn't recognize scrollbar size correctly.

Open
#37,596 0 comments 1 reaction 1 assignee View on GitHub

@mnajdova is already working on this.

Since Jun 15, 2023.

scope: menu type: bug
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:

  1. check MuiList's style on developer tools.
  2. you can find element.style is set to padding-right:0px.
    스크린샷 2023-06-15 오후 12 54 29
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)

스크린샷 2023-06-15 오후 12 54 40

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

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.