-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
[docs] Add @mui/base deprecation callout #45030
base: master
Are you sure you want to change the base?
Conversation
<Alert severity="warning"> | ||
<code>@mui/base</code> has been deprecated, please use the new Base UI{' '} | ||
<Link href={props.newComponentUrl}>{props.newComponentName} Component</Link> instead. | ||
</Alert> | ||
); | ||
} | ||
// console.log(props); | ||
return ( | ||
<Alert severity="warning"> | ||
<code>@mui/base</code> has been deprecated, if you're looking for an unstyled React | ||
component library try out <Link href="https://www.base-ui.com">Base UI</Link> | ||
</Alert> |
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.
Please suggest some exact wording if this seems too short (or too blunt) @samuelsycamore @colmtuite 🙏
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.
For this text, how about:
@mui/base
has been deprecated and replaced by Base UI. We strongly advise using the newer package instead.
and for the components that have a Base UI equivalent we can say:
@mui/base
has been deprecated and replaced by Base UI. Please use the Base UI {{Whatever}} component instead.
dcf6140
to
b3000e9
Compare
would it make sense to also deprecate the actual |
@@ -3,6 +3,8 @@ productId: base-ui | |||
title: React Radio Group component | |||
githubLabel: 'component: radio' | |||
waiAria: https://www.w3.org/WAI/ARIA/apg/patterns/radio/ | |||
newUrl: https://base-ui.com/react/components/radio |
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.
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.
Maybe we should just remove these pages altogether (since there are no plans for them in mui/base
)?
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.
A lot of them are actually done and exist in Base UI now (RadioGroup, ToggleGroup, Tooltip), maybe we can just repeat the deprecation message
For the ones that aren't (Drawer, Rating) it could be useful to keep the link to the GH issue if it's still open and exists in the Base UI repo
}) { | ||
if (props.newComponentUrl && props.newComponentName) { | ||
return ( | ||
<Alert severity="warning"> |
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.
I think this warrants a big scary red callout to make sure people don't ignore it
<Alert severity="warning"> | |
<Alert severity="error"> |
} | ||
// console.log(props); | ||
return ( | ||
<Alert severity="warning"> |
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.
<Alert severity="warning"> | |
<Alert severity="error"> |
<Alert severity="warning"> | ||
<code>@mui/base</code> has been deprecated, please use the new Base UI{' '} | ||
<Link href={props.newComponentUrl}>{props.newComponentName} Component</Link> instead. | ||
</Alert> | ||
); | ||
} | ||
// console.log(props); | ||
return ( | ||
<Alert severity="warning"> | ||
<code>@mui/base</code> has been deprecated, if you're looking for an unstyled React | ||
component library try out <Link href="https://www.base-ui.com">Base UI</Link> | ||
</Alert> |
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.
For this text, how about:
@mui/base
has been deprecated and replaced by Base UI. We strongly advise using the newer package instead.
and for the components that have a Base UI equivalent we can say:
@mui/base
has been deprecated and replaced by Base UI. Please use the Base UI {{Whatever}} component instead.
@@ -3,6 +3,8 @@ productId: base-ui | |||
title: React Radio Group component | |||
githubLabel: 'component: radio' | |||
waiAria: https://www.w3.org/WAI/ARIA/apg/patterns/radio/ | |||
newUrl: https://base-ui.com/react/components/radio |
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.
Maybe we should just remove these pages altogether (since there are no plans for them in mui/base
)?
6ee50a3
to
edb4154
Compare
We could probably do this as soon as this callout is deployed to docs? Marking the package as deprecated on npm can probably do any time though, will discuss this next week with the team~ |
I'll open a separate PR just for the |
:::warning | ||
The Base UI Drawer component isn't available yet, but you can upvote [this GitHub issue](https://github.com/mui/base-ui/issues/38) to see it arrive sooner. | ||
:::error | ||
@mui/base has been deprecated and has been replaced by Base UI. Please follow [this GitHub issue](https://github.com/mui/base-ui/issues/38) to track the development of the Drawer component. |
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.
copyedit nitpick: can drop the second "has been":
@mui/base has been deprecated and has been replaced by Base UI. Please follow [this GitHub issue](https://github.com/mui/base-ui/issues/38) to track the development of the Drawer component. | |
@mui/base has been deprecated and replaced by Base UI. Please follow [this GitHub issue](https://github.com/mui/base-ui/issues/38) to track the development of the Drawer component. |
<Alert severity="error"> | ||
@mui/base has been deprecated and has been replaced by Base UI. Please use the Base UI{' '} | ||
<Link href={props.newComponentUrl} color="error.800" underline="always"> | ||
{props.newComponentName} Component |
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.
nitpick: component doesn't need to be capitalized here
{props.newComponentName} Component | |
{props.newComponentName} component |
Part of #44910
Added a callout to all the
/base-ui/*
pages (example)Removed the "Planned"/"Preview" chips from the sidebar
Some previously "planned" components are done in the new Base UI, the callout is just repeated in the body e.g. RadioGroup
Some planned ones are still not yet done in the new package, the body keeps the link to the related GH issue but the text is changed a bit, e.g. Drawer
I have followed (at least) the PR section of the contributing guide.