[Kde-games-devel] Review Request: Kapman: Score information makes Game Area smaller

Omer F. USTA omerusta at gmail.com
Fri May 15 06:58:46 CEST 2009



> On 2009-05-06 06:04:22, Alexander Smirnov wrote:
> > As for "after patch" screenshot: I think values on a separate line do not look pretty.
> > As for the code: if possible, try to get rid of magic numbers 2,5,8

Could you please review the newer version of my patch ?
at ->  http://reviewboard.kde.org/r/702

Now it is not in game area and imho it looks much better.


- Omer F.


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


On 2009-05-03 10:20:21, Omer F. USTA wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/659/
> -----------------------------------------------------------
> 
> (Updated 2009-05-03 10:20:21)
> 
> 
> Review request for KDE Games.
> 
> 
> Summary
> -------
> 
> Score , Level and Life information was at the bottom of the game.
> These informations are making game area smaller after they added to
> screen.
> I have moved these values to unused right screen area thanks to this
> now after game starts svg graphics won't resize and makes game smaller
> and it looks much prettier.
> I think screen shots will explain it better than my explanation 
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdegames/kapman/gamescene.cpp 962958 
> 
> Diff: http://reviewboard.kde.org/r/659/diff
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> Before Patch - Begining
>   http://reviewboard.kde.org/r/659/s/113/
> Before Patch - While Playing
>   http://reviewboard.kde.org/r/659/s/114/
> After Patch
>   http://reviewboard.kde.org/r/659/s/115/
> 
> 
> Thanks,
> 
> Omer F.
> 
>



More information about the kde-games-devel mailing list