-
-
Notifications
You must be signed in to change notification settings - Fork 1.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] Make overview demo work in codesandbox #10661
Conversation
Deploy preview: https://deploy-preview-10661--material-ui-x.netlify.app/ |
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.
Yep this works better. Shouldn't be too hard to make the icon work as well, but not really a priority 🙂
@@ -17,9 +16,9 @@ function ProLabel({ children }) { | |||
return ( | |||
<Stack direction="row" spacing={0.5} component="span"> | |||
<Tooltip title="Included in Pro package"> | |||
<Link href="/x/introduction/licensing/#pro-plan"> | |||
<a href="https://mui.com/x/introduction/licensing/#pro-plan"> | |||
<span className="plan-pro" /> |
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 believe this class is not available in the codesandbox. To fully make the demo work in CSB, could it make sense to make a styled()
component for this instead? With absolute url to the image?
Still a character encoding issue in the codesandbox. I believe the Had the core repo open so already pre-back-ported (😄) it mui/material-ui#39424 |
Follow up of #10619
Preview: https://deploy-preview-10661--material-ui-x.netlify.app/x/react-date-pickers/