Column resize doesn't properly work on mobile about fixed-data-table-2 HOT 11 CLOSED

Resizing in mobile view or touch devices works in v1.2.x

In our opinion this is false. This is actually the exact reason why we are trying a migration towards v2, hoping to fix the behavior on mobile.

from fixed-data-table-2.

Hello! Thanks a lot for this fix! We confirm that we have tested and that the resize and reorder works now on our application (on mobile) with this your new prerelease version. We are looking forward for a release version.

from fixed-data-table-2.

Thanks a lot @pradeepnschrodinger

from fixed-data-table-2.

Thanks for filing.
This is a regression; Resizing in mobile view or touch devices works in v1.2.x but not in v2.0.x 😞

from fixed-data-table-2.

Ok. Thank you! But do you have any ideea why if

• I clone your project (on the master branch),
• I add on the ResizeExample.js for each ResizeCell touchEnabled={true}
• I run your examples (using yarn site-dev-server ),

The resize is working ok on mobile?
https://www.youtube.com/watch?v=ssChIDgGRtE

(Also the reorder column example is working ok on the locally checked out project with touchEnabled={true} )

from fixed-data-table-2.

Interesting.

We're no longer passing down the touchEnabled prop to the cell renderers within FDT.
But the resizer plugin <ResizeCell /> still expects it, making it not work for touch devices.

I add on the ResizeExample.js for each ResizeCell touchEnabled={true}

So a quick workaround is to explicity pass down the prop to the cell renderer from the user's side, like what you did.

The actual fix likely involves FDT passing it down instead, like what I see for v1.2.x.

But do you have any idea why if the resize is working ok on mobile?

I unfortunately can't figure out why there's a difference in behavior with the local webpack server vs the one in hosted in codesandbox.
Even the v1.2.x version doesn't seem to work in codesandbox.
cc: @AmanGupta2708 , @karry08 , @abhisheksingh2410

On a side note, I'm also seeing an issue where trying to resize via touch results in the table being scrolled in the background.
It would be nice to check if both of these issues are stemming out from the same underlying problem.

from fixed-data-table-2.

@daniela-mateescu, @cristian-spiescu , I put a potential fix for this at #706, with a couple of other touch related issues I saw along the way.

I did a quick round of smoke testing by forking your example (see https://codesandbox.io/s/fixed-data-table-2-touch-resizing-s22nn7) and it seems to work well.

The fix can be grabbed by specifying "fixed-data-table-2": "https://github.com/schrodinger/fixed-data-table-2#fix-touch-handlers-prerelease", in your package.json file.
It would be awesome if you guys can test it with your project, and let us know if it works.

from fixed-data-table-2.

Thanks a lot! We'll give it a shot Monday and get back to you. Have a nice weekend.

from fixed-data-table-2.

@pradeepnschrodinger do you have an idea when is a next version available, containg this fix? Maybe 2.0.3?

from fixed-data-table-2.

Thanks for the testing out the fix and letting us know it works.

The next release containing this fix is likely v2.0.3, and should be released within a day or two.
I'll comment here and on the PR once it's done.

from fixed-data-table-2.

Fix released with v2.0.3.

cc: @daniela-mateescu , @cristian-spiescu

from fixed-data-table-2.