google-fastbutton icon indicating copy to clipboard operation
google-fastbutton copied to clipboard

TouchStart propagation prevention breaks Swiper w/ nested buttons.

Open lpender opened this issue 12 years ago • 0 comments

Hi all,

Love your work thus far.

I've been independently using FastButton for a few months now on a responsive website I'm developing.

I use iDangerous swiper and often put FastButtons inside of the Swiper.

However, as you can imagine, Line 61 blocks the Swiping functionality.

60    this.FastButton.prototype.onTouchStart = function(event) {
61        event.stopPropagation ? event.stopPropagation() : (event.cancelBubble=true); 
62        this.touchEvents.push(addListener(this.element, 'touchend', this, this.useCapture));

I've been using Fastbutton for months with Line 61 commented, without any issues whatsoever...

So the big question: Is there a good argument for preventing the touchstart from propagating? Should it become an option? Could it perhaps be removed altogether?

Yours true -Lee

lpender avatar Oct 28 '13 20:10 lpender