Skip to content
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] Fix inline previews #3081

Merged
merged 2 commits into from
Nov 4, 2021

Conversation

DanailH
Copy link
Member

@DanailH DanailH commented Nov 3, 2021

Fixes #2990

Based on mui/material-ui#28215

The diff is so big because the previews need to be generated.

Preview -> https://deploy-preview-3081--material-ui-x.netlify.app/components/data-grid/columns/#column-definitions

@DanailH DanailH added the docs Improvements or additions to the documentation label Nov 3, 2021
@DanailH DanailH self-assigned this Nov 3, 2021
@DanailH DanailH added the bug 🐛 Something doesn't work label Nov 3, 2021
@@ -13,8 +13,8 @@
"export": "rimraf docs/export && next export --threads=3 -o export && yarn build-sw",
"icons": "rimraf public/static/icons/* && node ./scripts/buildIcons.js",
"typescript": "tsc -p tsconfig.json",
"typescript:transpile": "node scripts/formattedTSDemos",
"typescript:transpile:dev": "node scripts/formattedTSDemos --watch"
"typescript:transpile": "cross-env BABEL_ENV=development babel-node --extensions \".tsx,.ts,.js\" scripts/formattedTSDemos",
Copy link
Member

Choose a reason for hiding this comment

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

Why do you need to switch to babel-node here? I would recommend to stick to just node until it becomes absolutely necessary.

Copy link
Member Author

Choose a reason for hiding this comment

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

If I leave it as it was the script fails. I checked how it's done in the core and copied the approach.

Copy link
Member

@alexfauquette alexfauquette left a comment

Choose a reason for hiding this comment

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

I tested some pages on preview, seems to work well :)

@DanailH DanailH mentioned this pull request Nov 4, 2021
@DanailH DanailH merged commit b9627c8 into mui:next Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work docs Improvements or additions to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[docs] Inline previews are no longer working in DataGrid pages
3 participants