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

Option "step" documentation is ambiguous #638

Open
katapofatico opened this issue Feb 22, 2024 · 2 comments
Open

Option "step" documentation is ambiguous #638

katapofatico opened this issue Feb 22, 2024 · 2 comments

Comments

@katapofatico
Copy link

katapofatico commented Feb 22, 2024

Hello,

In order to swipe faster, I've been playing with the options parameter. In my case, I'm using length : 110 for left|right and length : 180 for top|bottom. The option step documentation says that it means how often new touch move will be generated. My first thought was: "The fewer steps, the faster swipe" (and I've double-checked the interpretation with an AI assistant), but I've checked that it's just the opposite. If I'm right, please could you clarify the documentation? BTW: it's an opportunity to copy it from the code to the README.

Thanks in advance for your super helpful library.

@dmtrKovalenko
Copy link
Owner

Fan fact: I also though it should work this way 😅

If you have this off your hand could you post maybe a video of different options swiping?

@amoshydra
Copy link

Not sure if this helps, but I happen to create a canvas to test the swipe events.

issue-638.mp4

Swiping right over a length of 960px

Step:

  • 960
  • 480
  • 240
  • 120
  • 60
  • 30
  • 15
  • 7.5
  • 3.75

reproduce repo: https://github.com/amoshydra/repro-dmtrKovalenko-cypress-real-events-i-swipes/blob/real-swipe-steps-issue-638/src/components/input/Canvas.cy.tsx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants