-
Notifications
You must be signed in to change notification settings - Fork 8
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
feat: Remove Beta status from Header and deprecate Mega Menu #1814
Conversation
@@ -20,4 +20,5 @@ const MegaMenuRoot = forwardRef( | |||
|
|||
MegaMenuRoot.displayName = 'MegaMenu' | |||
|
|||
/** @deprecated Use child components in Header instead. */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We probably want this deprecation to appear in the change log. Where should we list it, under ‘Features’?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I think so. It's a deprecation of a feature.
@@ -20,4 +20,5 @@ const MegaMenuRoot = forwardRef( | |||
|
|||
MegaMenuRoot.displayName = 'MegaMenu' | |||
|
|||
/** @deprecated Use child components in Header instead. */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I think so. It's a deprecation of a feature.
The Header component is no longer in Beta.
The Mega Menu component is deprecated. I’ve updated the Status Badge component to allow customizing the status label and making it red for a deprecation.