FormidableLabs/react-swipeable

Expose swipe start?

feketegy opened this issue ยท 4 comments

Why not expose the touchStart / mouseDown events so users can separate out the very first swipe, instead of connecting it to the onSwiping event's first property?

@feketegy sensible question, ๐Ÿค” . I'm curious what that could look like.

Any thoughts or ideas?

@hartzis My specific use case would be to set up some states on the very first swipe event, I'm using your package more like a dragging & dropping library :)

This is why I was relying on the first property being true, but it has this bug: #221

I think it would be cleaner this way to expose an onSwipeStart event rather to rely on the first property in the onSwiping callback.

Great package btw ๐Ÿ‘

@feketegy Great suggestion. I think onSwipeStart is simple and small enough to introduce.

Thoughts on #226 as a solution?

I totally missed this, awesome addition, thanks for this!