Conversation
Fixes #102 where animation-free dropdowns ended up with tons of Ripple objects endlessly and invisibly animating in requestAnimationFrames, though I'll admit it's not a principled fix, in that I'm not sure what the root cause of the bug is. This seems to be roughly in line with user expectations I think, as someone who doesn't want animations probably also doesn't want a ripple animation. I went this route rather than making `noink` work because the noink property isn't mapped anywhere, and the requestAnimationFrame leak would still occur for people using noAnimations.
|
@notwaldorf I would value your perspective on @rictic 's rationale for |
|
I think I am also a bit concerned that clicking creates endless ripples -- is it because they don't cancel? Does this happen if |
|
I didn't track down where the ripples were being created. Poking at the properties of the ripples that were created, it seemed like they weren't making any progress on each rAF, and the end point of their animation was beyond their stopping point. It didn't look related to PolymerElements/paper-ripple#76, as each click created a new ripple. In my tests I was looking at ~12 ripples. If we want a principled answer here I can track it down. |
Fixes #102 where animation-free dropdowns ended up
with tons of Ripple objects endlessly and invisibly animating in
requestAnimationFrames, though I'll admit it's not a principled fix,
in that I'm not sure what the root cause of the bug is.
This seems to be roughly in line with user expectations I think, as
someone who doesn't want animations probably also doesn't want a
ripple animation.
I went this route rather than making
noinkwork because the noinkproperty isn't mapped anywhere, and the requestAnimationFrame leak
would still occur for people using noAnimations.