1132105 - Add a "loop animation" button to the animation inspector panel

Status: NEW

(DevTools :: Inspector: Animations, enhancement, P3)

Description

[Patrick Brosset <:pbro>]

Most animations on websites only run once (usually sync'd to user events like scrolling or mouse movement), but that makes it hard to work on them with the browser devtools.
It would help if we could set any animation to repeat infinitely from the animation inspector so that it's then easy to tweak it from the rule-view.

Comment 1

[Patrick Brosset <:pbro>]

My first idea, based on my current knowledge of the waapi, for implementing this was to just use the fact that currentTime will become settable soon (bug 1072037), but I had a discussion with Brian Birtles about it:

"
Yes, currentTime would let you achieve this but not very cleanly. Jonathan is working on startTime now and currentTime should happen shortly after.

It would be easier if we had finishing behaviour (bug 1074630) since then you could detect the end of the animation more easily. (We also recently resolved to add finish events to the player interface which would make it easier still but I don't know that they will land in Q1.)
"

So, using the finish event seems like one option. But Brian Birtles also suggested a way to automatically repeat the animation by changing its iteration count:

"
To automatically repeat the animation you'd currently have to clone the animation and change its iteration-count like so:

  var mutableAnim = player.source.clone();
  mutableAnim.timing.iterations = Infinity;
  player.source = mutableAnim;

It will be a while before we support that however. I think we should probably move 'iterations' to the player interface so you can repeat an animation without modifying its source. That too, however, will take a while to happen.
"

Indeed, being able to set the animation to repeat without actually changing the source would be awesome.

Comment 2

[Julian Descottes [:jdescottes]]

Inspector bug triage. Filter on CLIMBING SHOES.