search

gridstack / gridstack.js

Build interactive dashboards in minutes.
https://gridstackjs.com
MIT License
6.9k stars 1.29k forks source link

Mobile: delay dragging widget vs page scroll #2781

Open JulienLecoq opened 2 months ago

JulienLecoq commented 2 months ago

Subject of the issue

Currently, the drag start immediately which can cause annoying drag start on mobile device when all you want is to move on the page.

Your environment

gridstack: "^10.3.1" gridstack-angular: "^0.6.0-dev"

adumesny commented 2 months ago

Honestly you should have an explicit edit mode in real apps to make any changes to grid which should be static otherwise. sent from phone. excuse any typ-oh!

On Tue, Sep 10, 2024, 3:44 AM Juliano @.***> wrote:

Subject of the issue

Currently, the drag start immediately which can cause annoying drag start on mobile device when all you want is to move on the page. Your environment

gridstack: "^10.3.1" gridstack-angular: "^0.6.0-dev"

— Reply to this email directly, view it on GitHub https://github.com/gridstack/gridstack.js/issues/2781, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA2VRZWNTTUZ4SAFF2YP4ADZV3ERTAVCNFSM6AAAAABN6OW6KOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGUYTMMBYGM2TEOI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

JulienLecoq commented 2 months ago

Yes, single touch point comes across and drag widget instead.

I have an explicit edit mode. I have a grid of photos similarly to Tinder and some others app like so:

grid_on_big_screen

On big screen, it's okay since we don't have to scroll, but if you're on a smaller screen, this can get annoying since you have to scroll to see the full grid:

grid_on_small_screen
JulienLecoq commented 2 months ago

I never used two fingers to scroll on a page.

I would say that this issue is not a big problem since this only occurs on this page. I don't have a grid with drag & drop anywhere else. Also, I could swap the position of the two first big pictures with the grid to temporarily solve this issue.

But I guess it's still a nice feature to have and some other people might need it in the future.

adumesny commented 2 months ago

Yeah 1 finger to page scroll and move items. I guess the only way to disembiguate would be to require user holding down on the widget long enough to turn into move mode similar to phone dashboard editing. Makes sense for mobile. Feature request if someone wants to donate for it. sent from phone. excuse any typ-oh!

On Tue, Sep 10, 2024, 9:01 AM Juliano @.***> wrote:

I never used two fingers to scroll on a page.

I would say that this issue is not a big problem since this only occurs on this page. I don't have a grid with drag & drop anywhere else. Also, I could swap the position of the two first big pictures with the grid to temporarily solve this issue.

But I guess it's still a nice feature to have and some other people might need it in the future.

— Reply to this email directly, view it on GitHub https://github.com/gridstack/gridstack.js/issues/2781#issuecomment-2341354558, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA2VRZT2RKM3KQKI5SWPAADZV4JVRAVCNFSM6AAAAABN6OW6KOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBRGM2TINJVHA . You are receiving this because you commented.Message ID: @.***>

JulienLecoq commented 2 months ago

Yes 👍🏻