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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[XGrid] Column resizing not working on touch devices #2007

Closed
2 tasks done
m4theushw opened this issue Jul 1, 2021 · 1 comment 路 Fixed by #2089
Closed
2 tasks done

[XGrid] Column resizing not working on touch devices #2007

m4theushw opened this issue Jul 1, 2021 · 1 comment 路 Fixed by #2089
Assignees
Labels
regression A bug, but worse

Comments

@m4theushw
Copy link
Member

  • The issue is present in the latest release.
  • I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 馃槸

Trying to resize a column on a touch device doesn't work.

ijcwULWI79

Expected Behavior 馃

It should allow to resize.

Context 馃敠

This is a regression from #1720. When the effect runs the ref below is undefined, then it doesn't add the listener.

https://github.com/mui-org/material-ui-x/blob/de37d4b92d0534bd541598d88ea0df25710de6ae/packages/grid/_modules_/grid/hooks/features/columnResize/useGridColumnResize.tsx#L313

Raised in #1989 (comment) and #1967 (comment).

@oliviertassinari
Copy link
Member

It looks like now is a great time for adding a test #537 (comment) 馃榿. It was broken in #1720

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression A bug, but worse
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants