-
-
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] Improve theme augmentation documentation #14115
Comments
apparently doing this import fixed the issue:
However, I don't think we should need to do that extra step as is either not documented |
Actually, per mui/material-ui#40996, I think that we should update the docs in MUI X and remove all mentions of TypeScript v3: https://github.com/search?q=org%3Amui+%22TypeScript+3.x%22&type=code. Seeing this: triggered me. Moved to #14239 |
If someone changes how we phrase the theme augmentation, please do the same for the other X components 🙏 |
createTheme
intellisense
I'll add this to the board, so we don't lose track of this! |
@MiguelG97: How did we do? Your experience with our support team matters to us. If you have a moment, please share your thoughts in this short Support Satisfaction survey. |
I see, my bad. in the "datagrid > style" section too |
https://mui.com/x/react-data-grid/style/#theme-header-and-pinned-sections right, yeah, this sounds wise 👍. I think that we can consider this the root problem. So updating this docs to also cover TypeScript should do it. |
createTheme
intellisense
Latest version
Steps to reproduce
Link to live example: (required)
Steps:
import { createTheme } from '@mui/material/styles';
The createTheme does not contain a field for MuiDataGrid
Current behavior
The createTheme does not contain a field for MuiDataGrid
Expected behavior
intellisense enable and field available
Context
MuiDataGrid field is not enable nor available
Your environment
Search keywords: MuiDataGrid
The text was updated successfully, but these errors were encountered: