Skip to content

Conversation

@atomiks
Copy link
Contributor

@atomiks atomiks commented Dec 8, 2025

Fixes #3349

This uses the same collision avoidance config that Popover + Tooltip use for submenus. That is, it allows the submenus to flip to the bottom or top if left/right don't fit by default, which doesn't apply to the top-level root menu (since it's undesirable to have that flip from bottom/top to left/right)

@atomiks atomiks added component: menu Changes related to the menu component. type: bug It doesn't behave as expected. labels Dec 8, 2025
@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 8, 2025

  • vite-css-base-ui-example

    pnpm add https://pkg.pr.new/mui/base-ui/@base-ui/react@3470
    
    pnpm add https://pkg.pr.new/mui/base-ui/@base-ui/utils@3470
    

commit: f6ad18e

@mui-bot
Copy link

mui-bot commented Dec 8, 2025

Bundle size report

Bundle Parsed size Gzip size
@base-ui/react 🔺+33B(+0.01%) 🔺+15B(+0.01%)

Details of bundle changes


Check out the code infra dashboard for more information about this PR.

@netlify
Copy link

netlify bot commented Dec 8, 2025

Deploy Preview for base-ui ready!

Name Link
🔨 Latest commit f6ad18e
🔍 Latest deploy log https://app.netlify.com/projects/base-ui/deploys/69395c018c0b780008da7747
😎 Deploy Preview https://deploy-preview-3470--base-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@atomiks atomiks marked this pull request as ready for review December 8, 2025 20:23
@atomiks atomiks force-pushed the fix/menu-submenu-clip branch from 948d242 to adefdd1 Compare December 9, 2025 09:41
@atomiks atomiks merged commit 4fafb90 into mui:master Dec 11, 2025
22 checks passed
@atomiks atomiks deleted the fix/menu-submenu-clip branch December 11, 2025 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: menu Changes related to the menu component. type: bug It doesn't behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[menu] sub popup clips the viewport

3 participants