> On 2009-02-23 11:43:12, Aaron Seigo wrote: > > btw, some general input on the animation: > > > > * it should probably be a little shorter. it feels a bit "slow" right now > > because of the time length. > > * the selected tab probably shouldn't be drawn (or should fade in along > > with the icon movement?) until the icons are in place > > > > otherwise, it is a pretty cute little thing indeed.
Thanks for the input. Will fix the animation today and upload the new diff. The background of the tabbar does an animation now. - Mohammad Mehdi ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/63/#review252 ----------------------------------------------------------- On 2009-02-14 00:58:27, Mohammad Mehdi Salem Naraghi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/63/ > ----------------------------------------------------------- > > (Updated 2009-02-14 00:58:27) > > > Review request for Plasma and Aaron Seigo. > > > Summary > ------- > > A little animation that mimics the behaviour of my sony erricsson handy > whenever I get to the main menu. The Icons in kickoff's tabbar are all placed > at the center of the tabbar before moving towards their intended location > when the menu is displayed. It's a hack as of now and doesn't even work when > the panel is placed on the right or left screen edge. But that's pretty easy > to fix if you guys like that behaviour. I am not sure, maybe the thing is > superfluous but hey, I like small useless animations. > > > Diffs > ----- > > trunk/KDE/kdebase/workspace/plasma/applets/kickoff/ui/tabbar.h 925803 > trunk/KDE/kdebase/workspace/plasma/applets/kickoff/ui/tabbar.cpp 925803 > > Diff: http://reviewboard.kde.org/r/63/diff > > > Testing > ------- > > the animation doesn't work (yet) if the panel is placed on the right or left > screen edge. But adding support for that is pretty straightforward. > > > Thanks, > > Mohammad Mehdi > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel