----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100521/#review1584 -----------------------------------------------------------
Please, revert this patch or fix it: * Currently single click doesn't seem to work * I don't think double click for reset is a good idea. - Davide On Feb. 2, 2011, 9:39 a.m., Romário Rios wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/100521/ > ----------------------------------------------------------- > > (Updated Feb. 2, 2011, 9:39 a.m.) > > > Review request for Plasma. > > > Summary > ------- > > This patch does some little changes to Timer plasmoid: > - Now it takes only one click to start, resume and pause it > - It now blinks when paused, making it quicker to determine if it's paused > or running > - Double-click resets timer > > > Diffs > ----- > > applets/timer/timer.h d191fa3 > applets/timer/timer.cpp a09aa4c > > Diff: http://git.reviewboard.kde.org/r/100521/diff > > > Testing > ------- > > As it's a simple modification, I think that it brings no bugs (except by the > fact that it says timeout when reseted with double-click). > > > Thanks, > > Romário > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
