Skip to content

Issues with devices that have a touchscreen and using a mouse #2413

Closed

Description

@ginobili

Subject of the issue

There is an issue with the advanced Demo two.hml with devices that have a touchscreen and using a mouse. There is an issue when you drag an item and then abort the drag. The drop placeholder still remains at the same place.

Your environment

Dell XPS 15 with touchscreen. Win 11 Pro with Chrome 116

Steps to reproduce

In the Advanced Demo two.hml Start dragging an element and then aborting the drag

gridstack.js._.Build.interactive.dashboards.in.minutes.-.Google.Chrome.2023-08-23.17-45-03.mp4

Expected behavior

Should have the same behaviour as regular desktop users

Activity

  1. Ruslan207 commented on Aug 24, 2023

    @Ruslan207
    Contributor

    Steps to reproduce

    In common laptop you can

    1. open Chrome devtools
    2. enter to emulation mode
    Screenshot 2023-08-25 at 00 40 58
    1. then choose "Desktop (touch)"
    Screenshot 2023-08-25 at 00 42 04
    1. refresh demo page (it will init const isTouch as true)
    2. then choose "Desktop"
    3. repeat dragging from video above
  2. adumesny commented on Aug 25, 2023

    @adumesny
    Member

    @Ruslan207 this is great! thank you for the steps too! learn something everyday as I didn't know you could combine them...

    I actually don't see "Desktop (touch)" or Desktop as a device actually (even if I try to add others)
    image

    OOps, I can create them. ok!
    image

  3. Ruslan207 commented on Aug 25, 2023

    @Ruslan207
    Contributor

    @adumesny select "Responsive" and click here
    Screenshot 2023-08-25 at 15 45 04

  4. added a commit that references this issue on Aug 25, 2023
  5. adumesny commented on Aug 25, 2023

    @adumesny
    Member

    fixed in next release (soon as I also need fixes)

  6. BananaAcid commented on Jan 1, 2024

    @BananaAcid
    1. refresh demo page (it will init const isTouch as true)

    oookaaaay ... wos wondering, if there is a bug ... could not find anything in the docs (except enabling resize-arrows) and thought touch was broken ... @adumesny I REALLY see a need for it to be added to the docs, boxed, underlined, bold and italic ;)

  7. adumesny commented on Jan 1, 2024

    @adumesny
    Member

    @BananaAcid you mean that resize handle is shown by default on mobile (isTouch=true) devices ?
    already has it's own section on the main page...
    https://github.com/gridstack/gridstack.js?tab=readme-ov-file#touch-devices-support

    Maybe that banana needs to do less acid :)

  8. BananaAcid commented on Jan 1, 2024

    @BananaAcid

    @adumesny no, just what to do, to be able to test it in the browser - and to make clear, it would work if one reloads the page. The only hint I could find for touch support was the handle related info in the docs/readme.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions