-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
[DataGrid] Make column resizing and autosizing available in Community plan #12420
[DataGrid] Make column resizing and autosizing available in Community plan #12420
Conversation
Deploy preview: https://deploy-preview-12420--material-ui-x.netlify.app/ Updated pages: |
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.
LGTM, MIT community is empowered more now 💪 🎉
One of the prominent highlights of the release this week + stable release announcements!
Nit: Might want to update here after the PR merge.
@@ -436,7 +440,11 @@ export const useGridColumnResize = ( | |||
fillerLeftRef.current = findGridElement(apiRef.current, 'filler--pinnedLeft'); | |||
fillerRightRef.current = findGridElement(apiRef.current, 'filler--pinnedRight'); | |||
|
|||
const pinnedPosition = apiRef.current.isColumnPinned(colDef.field); | |||
const pinnedPosition = apiRef.current.unstable_applyPipeProcessors( |
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.
👍
Oh wow, how did I miss this? This is huge, column resizing was literally the only reason why I wasn't using X grid in side projects. Great initiative 🚀 |
resizable={false} | ||
resizable |
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.
This looks wrong to me. Issue open: #13247
Closes #12358