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

Romário Rios luizromario at gmail.com
Wed Feb 2 18:55:15 CET 2011



> On Feb. 2, 2011, 4:31 p.m., Aaron J. Seigo wrote:
> > do you have a commit account, or do you need someone to commit for you? reason i ask is that i've merged your patch here and made some adjustments to it; if you have a commit account i can send you an updated version of your patch to polish further if you wish and then push, otherwise, i can push it for you.

I have no commit account. Push it.


- Romário


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


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/20110202/5ad29279/attachment.htm 


More information about the Plasma-devel mailing list