FormidableLabs / FormidableLabs/react-swipeable

Touch tap results in mouse events

Aperta
#231 9 commenti 3 reazioni 1 assegnatario Rivendicata da @hartzis Vedi su GitHub
bug 🐛 has example ✅ has solution ✅
Lingua principale
TypeScript
Stelle
2.1k
Fork
149
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

**Describe the bug**
Tapping in the component tracked by the handlers causes multiple handlers to be called

1. The `onTap` handler is called with event of type `touchend`
1. The `onSwiping` handler is called with some delta values
1. The `onTap` handler is called twice, both with event of type `mouseup`

**Steps or Sandbox to reproduce**
Observe the following codesandbox application console log when touch-tapping in the blue area.

https://codesandbox.io/s/react-swipeable-tap-bug-yrbyu

**Expected behavior**
WHEN both `trackMouse` and `trackTouch` options are set to `true`
GIVEN a single touch tap in the component
THEN only `onTap` handler is called, with one event of type `touchend`

**Device/Browser**
Tested on
* Dell XPS with touch screen, running Fedora 34, in Firefox, Chrome and Brave
* Samsung Galaxy S10, running Android 11, in Chrome and Brave

Sliding/swiping works very well in all the mentioned environments, but a single touch tap also gives the same result in all of them.

**Additional context**
The _react-swipeable_ hook as such is very time saving and smooth! Thanks for the great work!

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.