Review Request: Adding a chronometer to the fifteenPuzzle plasmoid

[Alex Raymond]

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

(Updated 2010-08-05 18:22:37.828812)


Review request for Plasma and Aaron Seigo.


Changes
-------

showMessage implemented! I think it looks alright :)
http://imagebin.ca/view/TrFK9o9.html


Summary
-------

This patch implements a chronometer for solving the puzzle. It starts as soon as it is shuffled, and automatically ends when the puzzle is sorted. A Plasma::Dialog is shown with the time elapsed after sorting out.


Diffs (updated)
-----

  /trunk/KDE//kdeplasma-addons/applets/fifteenPuzzle/src/fifteen.h 1159592 
  /trunk/KDE//kdeplasma-addons/applets/fifteenPuzzle/src/fifteen.cpp 1159592 
  /trunk/KDE//kdeplasma-addons/applets/fifteenPuzzle/src/fifteenPuzzle.h 1159592 
  /trunk/KDE//kdeplasma-addons/applets/fifteenPuzzle/src/fifteenPuzzle.cpp 1159592 

Diff: http://reviewboard.kde.org/r/4870/diff


Testing
-------

Probably it would only be testable if the position patch is applied, since the initial positions on the current revision create an unsolvable puzzle.


Thanks,

Alex

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100805/03adfce6/attachment.htm