----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3513/ -----------------------------------------------------------
(Updated 2010-04-22 22:26:21.798330) Review request for Plasma, Aaron Seigo, Marco Martin, igorto, and Adenilson Cavalcanti. Changes ------- Updated patch according to Marco Martin's review. Plasma::RotationAnimation still has some changes, but the interface now remains the same. This seems to help me find out that after Aaron Seigo's EasingAnimation class, we're doing double work on every animation which uses easingCurve()->valueForProgress() to calculate the delta. Also, Plasma::PendulumCurve is now properly exported from Plasma. A new example code for "Animation Example" plasmoid can be found in http://pastebin.com/cJtktUwR Summary (updated) ------- The Plasma::PendulumCurve class subclasses from Qt's QEasingCurve and implements a static function which simulates a "pendulum" effect by varying the progress value from 0 to 1, 1 to 0, 0 to -1 and -1 to 0 in a linear way. Together with this easing curve, some modifications were made on the Plasma::RotationAnimation to make the widget's rotation remain the last value given by updateEffectiveTime, and not a hard-coded one when the animation reaches the Stopped status. Diffs (updated) ----- trunk/KDE/kdelibs/plasma/CMakeLists.txt 1117000 trunk/KDE/kdelibs/plasma/animations/pendulumcurve.h PRE-CREATION trunk/KDE/kdelibs/plasma/animations/pendulumcurve.cpp PRE-CREATION trunk/KDE/kdelibs/plasma/animations/rotation.cpp 1117000 trunk/KDE/kdelibs/plasma/animations/rotation_p.h 1117000 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/cJtktUwR Thanks, Bruno _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel