Review Request 109526: Implement SH_Widget_Animate in KStyle

Kevin Ottens ervin at kde.org
Mon Mar 18 09:58:30 UTC 2013



> On March 18, 2013, 9:57 a.m., Kevin Ottens wrote:
> > Looks good, can become a "shit it" once the patch in Qt his qt5.git.

Ahem... s/shit/ship/ obviously. Now that was an "interesting" typo, I think I'll have to consult a shrink for finding a reason for that one. :-)


- Kevin


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


On March 16, 2013, 6:41 p.m., Àlex Fiestas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109526/
> -----------------------------------------------------------
> 
> (Updated March 16, 2013, 6:41 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Description
> -------
> 
> Implement SH_Widget_Animate in KStyle using the configuration we already have in kdeglobal.
> 
> I have NOT implemented detailed information about animations because after all this year only NoEffect and SimpleAnimation are used (kglobalsettings.cpp GraphicEffect), so what we want is a YES or NO instead of how much animation is required.
> 
> There is NO use of GradientEffects or ComplexAnimationEffects in all kdelibs or kde (grepped 204 repositories in KDE)
> 
> Review depends on:
> https://codereview.qt-project.org/#change,51166
> 
> 
> Diffs
> -----
> 
>   kdeui/kernel/kstyle.cpp 681b940 
> 
> Diff: http://git.reviewboard.kde.org/r/109526/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Àlex Fiestas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130318/d12627ea/attachment.html>


More information about the Kde-frameworks-devel mailing list