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

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 February 2nd, 2011, 9:39 a.m., Romário Rios wrote:

Review request for Plasma.
By Romário Rios.

Updated Feb. 2, 2011, 9:39 a.m.

Description 

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

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

Diffs

  • applets/timer/timer.h (d191fa3)
  • applets/timer/timer.cpp (a09aa4c)

View Diff