Interpolation of Calc values is broken

[dzbarsky]

In components/style/animation.rs, we have code for interpolating lengths.

The Interpolate impl for LengthOrPercentage and LengthOrPercentageOrAuto (and possibly others, wherever Calc is an enum option) should handle LengthOrPercentage::Calc by interpolating the absolute component and the percentage and then combining them.

E-easy? Happy to mentor.

[dzbarsky]

(This will be an issue after #7185 lands)

[notriddle]

I'd be happy to take this one.

[dzbarsky]

Great, I'll let you know when that PR lands.

[dzbarsky]

@notriddle you can work on this now if you're still interested

[notriddle]

Yup, still interested.

On Wed, Sep 9, 2015 at 8:56 PM David Zbarsky notifications@github.com
wrote:

@notriddle https://github.com/notriddle you can work on this now if
you're still interested

—
Reply to this email directly or view it on GitHub
#7284 (comment).

[notriddle]

I added something that I think will work (manual testing bears this out), but how should I add it to the test suite?

[dzbarsky]

I guess we don't have transitionend events yet but you could do a 0s-length transition and check that the final value matches a reference? Might need to rAF to get it to paint the end state.