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.

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

[Bug]: MenuButton has no method of scrolling menuitems when the list is long #16261

Closed
2 tasks done
crichtonibm opened this issue Apr 24, 2024 · 3 comments
Closed
2 tasks done

Comments

@crichtonibm
Copy link

Package

@carbon/react, @carbon/styles, @carbon/colors, @carbon/icons-react

Browser

Chrome, Safari, Firefox, Edge

Package version

1.54.0

React version

18.2.0

Description

When using a MenuButton, if the number of menuitems exceeds the height of the screen, there is no ability to scroll. The menu does try to place itself so it entirely fits on the screen, but if not, you cannot get to all of the menu items. If I use the scroll wheel of the mouse, it will scroll the page itself, but not the menu. Even when using the arrow keys to move the focus between menu items, the menu itself stays put while the blue selection disappears into the overflow.

Reproduction/example

https://stackblitz.com/edit/github-izkntc

Steps to reproduce

Create a MenuButton with many MenuItems, and try to get to / use the items 'off screen'

Suggested Severity

Severity 2 = User cannot complete task, and/or no workaround within the user experience of a given component.

Application/PAL

No response

Code of Conduct

@lee-chase
Copy link
Member

Simplest fix would be to allow a class on the popup. Adding max height to it produces a scroll bar.

@alisonjoseph alisonjoseph added severity: 3 https://ibm.biz/carbon-severity and removed severity: 2 https://ibm.biz/carbon-severity labels May 28, 2024
@alisonjoseph
Copy link
Member

@tay1orjones
Copy link
Member

Right now a max-height isn't in the design spec for MenuButton, so I don't think we'll add this as a default behavior right now. Applying max-height via your own styling/classes is the right approach

@tay1orjones tay1orjones closed this as not planned Won't fix, can't repro, duplicate, stale Jul 17, 2024
@github-project-automation github-project-automation bot moved this from ⏱ Backlog to ✅ Done in Design System Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

6 participants