Skip to content

[navigation-menu] Fix pointer-events lock when sweeping quickly across a trigger - #5454

Merged
atomiks merged 2 commits into
mui:masterfrom
edwardhorsey:fix/navigation-menu-sweep-stranded-pointer-events
Aug 10, 2026
Merged

[navigation-menu] Fix pointer-events lock when sweeping quickly across a trigger#5454
atomiks merged 2 commits into
mui:masterfrom
edwardhorsey:fix/navigation-menu-sweep-stranded-pointer-events

Conversation

@edwardhorsey

Copy link
Copy Markdown
Contributor

Fixes #5453.

When the pointer sweeps across a trigger too quickly for its content to open, the safe-polygon pointer-events: none lock is never released. Currently all the existing release paths only run after a menu has opened so in this case the list is left frozen.

This PR calls clearSafePolygonPointerEventsMutation on the trigger's mouseleave when no item is open.

@pkg-pr-new

pkg-pr-new Bot commented Aug 10, 2026

Copy link
Copy Markdown

commit: 483ec3a

@code-infra-dashboard

code-infra-dashboard Bot commented Aug 10, 2026

Copy link
Copy Markdown

Bundle size

Bundle Parsed size Gzip size
@base-ui/react 🔺+26B(+0.01%) 🔺+6B(0.00%)

Details of bundle changes

Performance

Total duration: 1,376.22 ms +46.90 ms(+3.5%) | Renders: 76 (+0) | Paint: 2,131.94 ms +32.61 ms(+1.6%)

Test Duration Renders
Popover mount (300 instances) 80.48 ms 🔺+16.71 ms(+26.2%) 1 (+0)

14 tests within noise — details


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

@netlify

netlify Bot commented Aug 10, 2026

Copy link
Copy Markdown

Deploy Preview for base-ui ready!

Name Link
🔨 Latest commit 483ec3a
🔍 Latest deploy log https://app.netlify.com/projects/base-ui/deploys/6a79adaa14bb3a0008c97276
😎 Deploy Preview https://deploy-preview-5454--base-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@atomiks atomiks added component: navigation menu Changes related to the navigation menu component. type: bug It doesn't behave as expected. labels Aug 10, 2026

@atomiks atomiks left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@atomiks
atomiks enabled auto-merge (squash) August 10, 2026 10:53
@atomiks
atomiks merged commit 7ddc954 into mui:master Aug 10, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[navigation menu] Sweeping across a trigger without opening the content locks the NavigationMenu

2 participants