1170159 - Timelines in the animation inspector should change color/get an icon when they are finished or paused

Status: NEW

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

Description

[Patrick Brosset <:pbro>]

The animation inspector can show several animations in parallel.
While some animations may still be running, others may be finished or paused.

With the current UI, that's easy to tell because each animation has its widget with a scrubber.

When the new UI (bug 1153271), all displayed animations share a common scrubber, so we should visually differentiate the animations that are finished or paused.

Comment 1

[Patrick Brosset <:pbro>]

In fact, if the timeline displays, says, 2 animations, 1 of which is paused and the other is running, then the one that's paused should start moving along with the scrubber. So that if it's resumed again later, it is shown at the right place in the timeline.

Consider the following timeline where:
- A1 and A2 are animations
- the vertical line is the current scrubber position

            V
            |
[------A1---|---]
     [------|----A2-----------]
            |

If at this time A1 gets paused, and A2 still runs, after some time passes, instead of getting this:

                    V
                    |
[------A1-------]   |
     [-----------A2-|---------]
                    |

we should get this:

                    V
                    |
        [------A1---|---]
     [-----------A2-|---------]
                    |

Because A1 is paused, and if it gets resumed right then, that's where it should be displayed.

We could either make A1 move along with time, or just freeze it, change its color to show that it's paused, and then, if resumed, move it to its new location, whatever works best.

Comment 2

[Patrick Brosset <:pbro>]

Note that bug 1180134 adds color-coding to animations, so simply using a different color for paused animations might not be the best.
Also note that for now, no events are sent to the front-end when an animation gets paused, so this will have to wait for bug 1194639 to land first.

Comment 3

[Julian Descottes [:jdescottes]]

Inspector bug triage. Filter on CLIMBING SHOES.

Comment 4

[Patrick Brosset <:pbro>]

Inspector bug triage (filter on CLIMBING SHOES).