[Kde-games-devel] Review Request 109689: Change the cursor between game states.

[Albert Astals Cid]

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

Ship it!


Once you fix the little spaces VS tabs issue below feel free to commit to master, seems pretty sage to me


gamewidget.h
<http://git.reviewboard.kde.org/r/109689/#comment22242>

    Fix spaces vs tab


- Albert Astals Cid


On March 25, 2013, 6:46 p.m., Roney Gomes wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109689/
> -----------------------------------------------------------
> 
> (Updated March 25, 2013, 6:46 p.m.)
> 
> 
> Review request for KDE Games.
> 
> 
> Description
> -------
> 
> Currently, the mouse cursor keeps the same between game states, i.e., if the game is paused or the stage is cleared then the "in game" cursor will still be used.
> 
> I think that whenever the game is paused, even better, whenever there's an overlay on screen the mouse cursor shall be the system's default. The vertical and horizontal arrows shall be used only when the game is running.
> 
> Following the least responsibility principle, I've made a new method just to handle cursor variations. It checks the game state then updates the cursor accordingly.
> 
> 
> Diffs
> -----
> 
>   gamewidget.h 14abeaf 
>   gamewidget.cpp d808e46 
> 
> Diff: http://git.reviewboard.kde.org/r/109689/diff/
> 
> 
> Testing
> -------
> 
> I've played the game and then checked how the cursor changed between all game states: running, paused, between levels, game over etc.
> 
> 
> Thanks,
> 
> Roney Gomes
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20130326/36e0ac85/attachment.html>