1239298 - Delays don't have the right size in the animation timeline

Status: RESOLVED FIXED

(DevTools :: Inspector: Animations, defect)

Description

[Patrick Brosset <:pbro>]

In bug 1171863 the whole UI of the timeline was changed from pixel units to percentages, so that it would resize when the sidebar was resized.

Unfortunately, some elements of the timeline are nested into each other, so 50% of something that's already 50% of the timeline width will end up being 25%.
This was overlooked in bug 1171863 as I simply changed px to %.

We need to rework the way the elements are nested in the timeline to avoid this.
An animation iteration 1% wide should be as wide as an animation delay of 1%.

Comment 1

[Patrick Brosset <:pbro>]

Attachment: MozReview Request: Bug 1239298 - Make sure percentage width of iterations and delays in the timeline are relative to the same parent

Review commit: https://reviewboard.mozilla.org/r/30685/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/30685/

Comment 2

[Patrick Brosset <:pbro>]

Comment on attachment 8707420 [details]
MozReview Request: Bug 1239298 - Make sure percentage width of iterations and delays in the timeline are relative to the same parent

Mike is on PTO these days, perhaps Tom could review this?

Comment 3

[Patrick Brosset <:pbro>]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=c8022c48b5f0

Comment 4

[Tom Tromey :tromey]

Comment on attachment 8707420 [details]
MozReview Request: Bug 1239298 - Make sure percentage width of iterations and delays in the timeline are relative to the same parent

https://reviewboard.mozilla.org/r/30685/#review27689

This looks reasonable to me.  I noticed, though, that there is no associated test.  If this is impractical, then it's ok; but otherwise something is needed.

Comment 5

[Patrick Brosset <:pbro>]

(In reply to Tom Tromey :tromey from comment #4)
> This looks reasonable to me.  I noticed, though, that there is no associated
> test.  If this is impractical, then it's ok; but otherwise something is
> needed.
So we do have a few tests that ensure delay elements are positioned correctly in the timeline. So at least, if these still pass, there won't be any regressions related to where and how wide delays are shown.

Now, there's been quite a bunch of HTML restructuring and CSS changes here, so there could theoritically be regressions related to the visual aspect of delay elements, but so far in devtools we've never had any sort of automated visual tests, so I don't think I can add anything here.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/436190769b9e

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/436190769b9e

Comment 8

[Mayur Patil]

[bugday-20160323]

Status: RESOLVED,FIXED -> UNVERIFIED

Comments:
STR: Not clear.
Developer specific testing

Component: 
Name			Firefox
Version			46.0b9
Build ID		20160322075646
Update Channel          beta
User Agent		Mozilla/5.0 (Windows NT 6.1; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0
OS			Windows 7 SP1 x86_64

Expected Results: 
Developer specific testing

Actual Results: 
As expected