search

PolymerElements / paper-ripple

Adds a Material Design ripple effect to UI elements
https://webcomponents.org/element/PolymerElements/paper-ripple
57 stars 27 forks source link

transitionend can be fired twice #75

Open MartinMoizard opened 8 years ago

MartinMoizard commented 8 years ago

When controlling a paper-ripple through its API (downAction and upAction), the transitionend event can be fired more than once.

The issue can be reproduced using the following code: https://jsfiddle.net/2q3fqehz/34/

My question is the following: in upAction of paper-ripple.html, why not replacing

this.animate();

by

if (!this._animating) {  
  this.animate();  
}

?