[Kde-games-devel] Review Request: Bug 308380 - Canceling change of difficulty level restarts the current game.

[Albert Astals Cid]

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


Maybe hook ourselves to currentLevelChanged instead of selectedLevelChanged ?

- Albert Astals Cid


On Oct. 26, 2012, 12:58 p.m., Roney Gomes wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107053/
> -----------------------------------------------------------
> 
> (Updated Oct. 26, 2012, 12:58 p.m.)
> 
> 
> Review request for KDE Games.
> 
> 
> Description
> -------
> 
> As described in the bug tracker, the game restarts even if the player decides to cancel the current selection. In the attached patch, the game verifies whether the player has decided to select a different level or not.
> 
> 
> This addresses bug 308380.
>     http://bugs.kde.org/show_bug.cgi?id=308380
> 
> 
> Diffs
> -----
> 
>   src/mainwindow.cpp a7cceeb 
>   src/playfield.h 56934dc 
>   src/playfield.cpp 61c0ac6 
> 
> Diff: http://git.reviewboard.kde.org/r/107053/diff/
> 
> 
> Testing
> -------
> 
> Changed the difficulty level and then canceled the current selection.
> Changed the difficulty level and then accepted the current selection.
> 
> 
> Thanks,
> 
> Roney Gomes
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20121101/1755a57b/attachment-0001.html>