<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://svn.reviewboard.kde.org/r/6931/">http://svn.reviewboard.kde.org/r/6931/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On April 1st, 2012, 9:16 p.m., <b>Albert Astals Cid</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Some comments: 
 * Please make so that the font change is applied immediately not on new game start
 * Let the user choose the font size in points not in an arbitrary 0 to 10 scale</pre>
 </blockquote>




 <p>On April 2nd, 2012, 5:14 a.m., <b>Arjun Basu</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">On making the changes appear immediately , the rows take time to resize to the changed font size, which doesn't look so good visually. Should I still implement that.
I didn't understand your second point completely, regarding choosing the font size in points, are you implying that the user should get to choose the actual point size of the font?</pre>
 </blockquote>





 <p>On April 2nd, 2012, 4:41 p.m., <b>Albert Astals Cid</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">How much time is "time"?

And yes, the user should get to choose the point size, it's something they've been doing for ages in word processors so feels much more natural than an arbitrary 0 to 10 scale</pre>
 </blockquote>





 <p>On April 2nd, 2012, 4:49 p.m., <b>Arjun Basu</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Actually, I kind of said it wrong.Actually on increasing the font size instantly, the rows don't increase in height to accommodate the change in font size until they are clicked.
And what is the default point size for kiriki? </pre>
 </blockquote>





 <p>On April 2nd, 2012, 5:03 p.m., <b>Albert Astals Cid</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The default point size for kiriki is the default point size of the QFont of the system</pre>
 </blockquote>





 <p>On April 3rd, 2012, 4:02 a.m., <b>Arjun Basu</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Ok I will make the change and you didn't say anything whether I should do the font change immediately or on game restart</pre>
 </blockquote>








</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Immediately, fixing the issue that you need to click on them for the resize to happen.</pre>
<br />








<p>- Albert</p>


<br />
<p>On March 31st, 2012, 9:30 p.m., Arjun Basu wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for KDE Games.</div>
<div>By Arjun Basu.</div>


<p style="color: grey;"><i>Updated March 31, 2012, 9:30 p.m.</i></p>






<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I have tested it and it works fine</pre>
  </td>
 </tr>
</table>




<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>/trunk/KDE/kdegames/kiriki/src/configPage.ui <span style="color: grey">(1282954)</span></li>

 <li>/trunk/KDE/kdegames/kiriki/src/kiriki.cpp <span style="color: grey">(1282954)</span></li>

 <li>/trunk/KDE/kdegames/kiriki/src/kiriki.kcfg <span style="color: grey">(1282954)</span></li>

 <li>/trunk/KDE/kdegames/kiriki/src/row.h <span style="color: grey">(1282954)</span></li>

 <li>/trunk/KDE/kdegames/kiriki/src/scores.h <span style="color: grey">(1282954)</span></li>

 <li>/trunk/KDE/kdegames/kiriki/src/scores.cpp <span style="color: grey">(1282954)</span></li>

</ul>

<p><a href="http://svn.reviewboard.kde.org/r/6931/diff/" style="margin-left: 3em;">View Diff</a></p>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>

<div>

 <a href="http://svn.reviewboard.kde.org/r/6931/s/629/"><img src="http://svn.reviewboard.kde.org/media/uploaded/images/2012/03/31/PreferencesDialog_400x100.jpeg" style="border: 1px black solid;" alt="The New Preferences Dialog" /></a>

 <a href="http://svn.reviewboard.kde.org/r/6931/s/630/"><img src="http://svn.reviewboard.kde.org/media/uploaded/images/2012/03/31/Canvas_400x100.jpeg" style="border: 1px black solid;" alt="The Game Canvas after increasing font size" /></a>

</div>


  </td>
 </tr>
</table>








  </div>
 </body>
</html>