[Kde-games-devel] Review Request: Increase the Row Height of Kiriki

Arjun Basu arjunkbasu at gmail.com
Fri Apr 6 14:32:28 UTC 2012


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

(Updated April 6, 2012, 2:32 p.m.)


Review request for KDE Games.


Changes
-------

I changed it, now it takes the parameters as the topleft and bottomright of the canvas and redraws it at one go, instead of iterating over the rows


Description
-------

This is in response to the feature request https://bugs.kde.org/show_bug.cgi?id=225582. Now, there is a provision of increasing the font size and as a result the row height increases too. The value of the font size in the preferences can take values from 0 to 10. O being the default font size.


Diffs (updated)
-----

  /trunk/KDE/kdegames/kiriki/src/configPage.ui 1282954 
  /trunk/KDE/kdegames/kiriki/src/kiriki.cpp 1282954 
  /trunk/KDE/kdegames/kiriki/src/kiriki.kcfg 1282954 
  /trunk/KDE/kdegames/kiriki/src/row.h 1282954 
  /trunk/KDE/kdegames/kiriki/src/scores.h 1282954 
  /trunk/KDE/kdegames/kiriki/src/scores.cpp 1282954 

Diff: http://svn.reviewboard.kde.org/r/6931/diff/


Testing
-------

I have tested it and it works fine


Screenshots
-----------

The New Preferences Dialog
  http://svn.reviewboard.kde.org/r/6931/s/629/
The Game Canvas after increasing font size
  http://svn.reviewboard.kde.org/r/6931/s/630/


Thanks,

Arjun Basu

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


More information about the kde-games-devel mailing list