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

[Menu] Fix openOnHover issues #1191

Merged
merged 14 commits into from
Jan 8, 2025
Merged

Conversation

atomiks
Copy link
Contributor

@atomiks atomiks commented Dec 20, 2024

Closes #1180

Includes data-instant style hook for Menu

@atomiks atomiks added the component: menu This is the name of the generic UI component, not the React module! label Dec 20, 2024
@mui-bot
Copy link

mui-bot commented Dec 20, 2024

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
@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
@@ -18,6 +18,10 @@
"data-closed": {
"description": "Present when the menu is closed."
},
"data-instant": {
"description": "Indicates the instant type of the menu popup.",
Copy link
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
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
23 checks passed
@atomiks atomiks deleted the fix/menu-open-on-hover branch January 8, 2025 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: menu This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Menu] openOnHover issues
5 participants