<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>
<p>On April 3rd, 2012, 9:01 a.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;">Immediately, fixing the issue that you need to click on them for the resize to happen.</pre>
</blockquote>
<p>On April 3rd, 2012, 4:58 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;">I am working on the resize thing, for that I think I need to redraw the game canvas without destroying the existing data, any idea how I can do that. I can delete the m_scores object and reinitialize it but that destroys the existing play state </pre>
</blockquote>
<p>On April 3rd, 2012, 5:05 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;">One thing can be done though, the canvas resizes itself when a new game is started. Can we give the user notification that the changes to the font size will be made from the next game onwards?</pre>
</blockquote>
<p>On April 3rd, 2012, 5:14 p.m., <b>Frederik Schwarzer</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;">You could try to find out what makes it resize and trigger that when settings have changed.</pre>
</blockquote>
<p>On April 3rd, 2012, 6:04 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;">It is good that the resizing is done after applying not on next game since it's actually "easy". I know how to do it but i'm not going to tell you since the point of JJ is for you to learn Qt/KDE, so i'm just going to give you a few pointers. Look at what class m_scores is and look at which method it has that may be useful for you, specially having into account in which method you are returning which is the font that should be used</pre>
</blockquote>
<p>On April 5th, 2012, 7:15 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;">I figured out the method in the scores class. I think it is this QVariant scores::data(const QModelIndex &index, int role). But I am having some problems regarding which parameter to pass as the role parameter. Using the kError() output I found that it is called multiple times. If you could provide a little help here, It would be great.</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;">You got it backwards, the data() function is what you are returning in your model, you don't need to call data(), since calling data() will do nothing (note that data() is const, i.e. basically a getter). What you need to do is inform those calling data() that your internal data has changed.</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>