Review Request: Added Plasma::Animator::PendulumAnimation

[Bruno Abinader]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3513/
-----------------------------------------------------------

(Updated 2010-04-08 16:18:21.798505)


Review request for Plasma, igorto and Adenilson Cavalcanti.


Changes
-------

Due to more talks about how the pendulum animation should work, we decided to transform it into a new kind of animation (PendulumAnimation), thus not altering the behavior of the already existing animations. These can be later mixed up using an animation group.


Summary (updated)
-------

The Plasma::Animator::PendulumAnimation class was created to perform a "pendulum" animation effect on the target widget, using the following properties:
- axis: Axis in which the pendulum animation occurs (Qt::XAxis, Qt::YAxis or Qt::ZAxis);
- angle: Maximum angle used as boundaries for the pendulum animation (eg. 45 degrees);
- pivotPos: Pendulum pivot position, based on the target widget's center position (eg. QPointF(0,0) means that the pivot position is the target widget's center position);
- isDecremental: A boolean used to set if the animation angle shall decrement in a proportional way to the number of loop counts;

This new animation class:
a) can be part of an animation group;
b) uses a settable easing curve;


Diffs (updated)
-----

  trunk/KDE/kdelibs/plasma/animations/pendulum_p.h PRE-CREATION 
  trunk/KDE/kdelibs/plasma/animator.h 1112209 
  trunk/KDE/kdelibs/plasma/animator.cpp 1112209 
  trunk/KDE/kdelibs/plasma/CMakeLists.txt 1112209 
  trunk/KDE/kdelibs/plasma/animations/pendulum.cpp PRE-CREATION 

Diff: http://reviewboard.kde.org/r/3513/diff


Testing (updated)
-------

An example code was added to the "plasma applet animation example" (in kdeexamples/plasma/c++/plasmoid/animation). That code will be later commited, after this request gets successfully merged. You can grab the example code here: http://pastebin.com/wfStG2p8


Thanks,

Bruno