-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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 Toolpad Core template #44415
base: master
Are you sure you want to change the base?
[docs] Add Toolpad Core template #44415
Conversation
Netlify deploy previewhttps://deploy-preview-44415--material-ui.netlify.app/ Bundle size report |
This relates to the discussion in https://github.com/mui/mui-private/pull/13, https://github.com/mui/mui-private/issues/648, and https://github.com/mui/mui-private/issues/569. |
href: '/material-ui/getting-started/templates/dashboard/', | ||
source: `${sourcePrefix}/docs/data/material/getting-started/templates/dashboard`, | ||
hasDarkMode: true, | ||
}, | ||
{ | ||
title: translatation('marketingPageTitle'), | ||
description: translatation('marketingPageDescr'), | ||
title: translation('dashboardToolpadTitle'), |
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.
Interleaving different types of content feels like a big no-no.
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.
Do you mean we should create a separate section for Toolpad Core templates on this page, or create a separate page for them, or do you mean that the Toolpad-specific copy should be stored separately from the rest of the content (not in translations.json
?)
externalInput
URL when the above is merged and the Toolpad docs website is deployedPreview: https://deploy-preview-44415--material-ui.netlify.app/material-ui/getting-started/templates/