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

[DataGrid] iOS freeze on drag #443

Closed
2 tasks done
oliviertassinari opened this issue Oct 15, 2020 · 6 comments
Closed
2 tasks done

[DataGrid] iOS freeze on drag #443

oliviertassinari opened this issue Oct 15, 2020 · 6 comments
Labels
bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module!

Comments

@oliviertassinari
Copy link
Member

oliviertassinari commented Oct 15, 2020

  • 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 😯

stuck

Expected Behavior 🤔

Doesn't get stuck

stuck ok

(react-virtual)

Your Environment 🌎

Tech Version
Material-UI X v4.0.0-alpha.6
Browser iOS Safari 14.0
@oliviertassinari oliviertassinari added status: waiting for maintainer These issues haven't been looked at yet by a maintainer bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module! and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Oct 15, 2020
@lenolib
Copy link

lenolib commented Aug 27, 2021

This is big blocker for us. It's easy to get lost in the table when scrolling on iOS and it jumps
https://user-images.githubusercontent.com/1769273/131104324-dde85395-be2f-45c5-ba66-a81bba89db81.mp4

@oliviertassinari
Copy link
Member Author

oliviertassinari commented Aug 31, 2021

@lenolib Thanks for the feedback. Please upvote this issue to give it more weight.

We have #1933 to gather the constraint on what we need to do next for the virtualization. I suspect that this issue means that we have to give up on the current virtualization logic (that is not relying on the native scroll behavior), but to dive deeper and challenge.

@m4theushw
Copy link
Member

@oliviertassinari Could you check if #2673 fixed this issue? The new virtualization logic uses the natural scroll.

@DanailH
Copy link
Member

DanailH commented Nov 19, 2021

Just checked it - it works as expected - tested it on iPhone 12, IOS 15.1.1. It works both on Chrome and Safari.

@oliviertassinari
Copy link
Member Author

Same, it seems fixed one iPhone 13, iOS 15.0.0.

@m4theushw However we have a different issue, see the header:

iOS

I can reproduce the same issue on AG Grid, it's not sure if we can fix this.

@m4theushw
Copy link
Member

However we have a different issue, see the header:

This is the trade-off of using the native scroll. See #2673 (comment) for an explanation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module!
Projects
None yet
Development

No branches or pull requests

5 participants