Review Request: Added "pendulum" effect for plasma animations

Marco Martin notmart at gmail.com
Thu Apr 8 19:03:17 CEST 2010


On Wednesday 07 April 2010, Aaron J. Seigo wrote:
> On April 7, 2010, you wrote:
> > This patch generated a discussion which ended up on a cooler idea (thanks
> > notmart, Savago and igorto!) that is providing two properties on the
> > Animation class in order to set up the pendulum behavior ("attenuation"
> > for the amount of delta value decrement between each bounce step and
> > "bounceCenter" to define from which point the target widget shall have
> > the same distance throughout the whole animation).
> 
> this is just an easing curve, no? only it's hardcoded into Animation. and
> so every time we'd want another behaviour, we'd end up harcoding another
> curve method into Animation?
> 
> Animation alreay has support for QEasingCurve. the attached patch makes it
> easier to use (with an example of how Rotation was modified; all the
> classes would need this modification though.)

yes, i like the patch, pretty straightforward.

> then this pedulum behaviour could become a QEasingCurve implementation. the
> next question becomes: do we want to offer a Animator::easingCurve(..)
> method?

this would be for js bindings, right? so we wouldn't have to do bindings on 
every qt easingcurve? (and all eventual ours)
at least that's the only point i would see it useful to be exported there

Cheers,
Marco Martin


More information about the Plasma-devel mailing list