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