Review Request: Timer: one click to pause/resume, blinks when paused.

Davide Bettio davide.bettio at kdemail.net
Tue Feb 22 17:56:02 CET 2011


-----------------------------------------------------------
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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110222/837318c1/attachment.htm 


More information about the Plasma-devel mailing list