Skip to content

[Menu] Group and Separator components#535

Merged
michaldudak merged 16 commits intomui:masterfrom
michaldudak:menu-subcomponents
Sep 27, 2024
Merged

[Menu] Group and Separator components#535
michaldudak merged 16 commits intomui:masterfrom
michaldudak:menu-subcomponents

Conversation

@michaldudak
Copy link
Member

@michaldudak michaldudak commented Aug 5, 2024

Created the Group and Separator components for the Menu.

Closes #70

@michaldudak michaldudak added component: menu Changes related to the menu component. type: new feature Expand the scope of the product to solve a new problem. labels Aug 5, 2024
@mui-bot
Copy link

mui-bot commented Aug 5, 2024

Netlify deploy preview

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

Generated by 🚫 dangerJS against 3a9d75d

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Sep 5, 2024
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Sep 6, 2024
@michaldudak michaldudak marked this pull request as ready for review September 6, 2024 13:26
@colmtuite colmtuite requested a review from vladmoroz September 9, 2024 08:55
Comment on lines +18 to +21
const MenuGroup = React.forwardRef(function MenuGroup(
props: MenuGroup.Props,
forwardedRef: React.ForwardedRef<Element>,
) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Can Menu groups have labels?

Copy link
Member Author

Choose a reason for hiding this comment

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

Added MenuGroupLabel component.

michaldudak and others added 2 commits September 18, 2024 11:13
Co-authored-by: atomiks <cc.glows@gmail.com>
Signed-off-by: Michał Dudak <michal.dudak@gmail.com>
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Sep 19, 2024
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Sep 19, 2024
@colmtuite
Copy link
Contributor

@michaldudak Label needs to be added to the Anatomy section code.

@michaldudak
Copy link
Member Author

Done.

@michaldudak michaldudak merged commit af5339d into mui:master Sep 27, 2024
@michaldudak michaldudak deleted the menu-subcomponents branch September 27, 2024 10:13
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: new feature Expand the scope of the product to solve a new problem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[separator] Implement Separator

4 participants