Skip to content

[menu] Fix openOnHover issues#1191

Merged
atomiks merged 14 commits into
mui:masterfrom
atomiks:fix/menu-open-on-hover
Jan 8, 2025
Merged

[menu] Fix openOnHover issues#1191
atomiks merged 14 commits into
mui:masterfrom
atomiks:fix/menu-open-on-hover

Conversation

@atomiks

@atomiks atomiks commented Dec 20, 2024

Copy link
Copy Markdown
Contributor

Closes #1180

Includes data-instant style hook for Menu

@atomiks atomiks added the component: menu Changes related to the menu component. label Dec 20, 2024
@mui-bot

mui-bot commented Dec 20, 2024

Copy link
Copy Markdown

Netlify deploy preview

https://deploy-preview-1191--base-ui.netlify.app/

Generated by 🚫 dangerJS against 8e06b2d

@atomiks atomiks marked this pull request as draft December 20, 2024 06:05
Comment thread packages/react/src/popover/root/usePopoverRoot.ts Outdated
@github-actions github-actions Bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Dec 23, 2024
@atomiks atomiks force-pushed the fix/menu-open-on-hover branch from e753d95 to 3d9ea80 Compare December 26, 2024 03:51
@github-actions github-actions Bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Dec 26, 2024
@atomiks atomiks marked this pull request as ready for review December 26, 2024 04:11
@github-actions github-actions Bot added PR: out-of-date The pull request has merge conflicts and can't be merged. and removed PR: out-of-date The pull request has merge conflicts and can't be merged. labels Jan 7, 2025
"description": "Present when the menu is closed."
},
"data-instant": {
"description": "Indicates the instant type of the menu popup.",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

TBH this description doesn't help much to understand what this attribute is for.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yep, agreed. The one for Popover is the same but I can update both of them.

@github-actions github-actions Bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Jan 8, 2025
@atomiks atomiks force-pushed the fix/menu-open-on-hover branch from 9de2385 to d12cefb Compare January 8, 2025 07:33
@github-actions github-actions Bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Jan 8, 2025
@atomiks atomiks merged commit 1eec235 into mui:master Jan 8, 2025
@atomiks atomiks deleted the fix/menu-open-on-hover branch January 8, 2025 08:50
@oliviertassinari oliviertassinari changed the title [Menu] Fix openOnHover issues [menu] Fix openOnHover issues May 23, 2026
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[menu] openOnHover issues

6 participants