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] Replace @mui/styles in x-data-grid-generator #3560
Conversation
0eb5319
to
64f28f9
Compare
64f28f9
to
0a2da36
Compare
); | ||
const Value = styled('div')(({ theme }) => ({ | ||
width: '100%', | ||
height: '100%', |
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.
I did not read all of them. I will finish the reading tomorow
packages/grid/x-data-grid-generator/src/renderer/renderEditContractType.tsx
Outdated
Show resolved
Hide resolved
const StyledAutocomplete = styled(Autocomplete)(({ theme }) => ({ | ||
[`& .${autocompleteClasses.inputRoot}`]: { | ||
...theme.typography.body2, | ||
padding: '1px 0', | ||
height: '100%', | ||
'& input': { | ||
padding: '0 16px', | ||
height: '100%', | ||
}, | ||
}, | ||
})); |
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 don't know why, but the autocompleteClasses.root
also needs to have height: "100%"
. Otherwise the Autocomplete component does not fill the cell
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.
Fixed in 1420d76.
packages/grid/x-data-grid-generator/src/renderer/renderEditCurrency.tsx
Outdated
Show resolved
Hide resolved
packages/grid/x-data-grid-generator/src/renderer/renderEditStatus.tsx
Outdated
Show resolved
Hide resolved
Just finished reading. The other files look good to me :) |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
These are the results for the performance tests:
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Part of #2991
After this PR there will only a few usages of
@mui/styles
remaining.Preview: https://deploy-preview-3560--material-ui-x.netlify.app/components/data-grid/demo/
Preview: https://deploy-preview-3560--material-ui-x.netlify.app/storybook/?path=/story/datagridpro-test-rows--edit-rows-control