<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://git.reviewboard.kde.org/r/109885/">http://git.reviewboard.kde.org/r/109885/</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 6th, 2013, 10 p.m. UTC, <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;">Besides broken spacing, i think it'd make sense having an option to toggle between the modes, people may like the current behaviour mroe even if it's irreal</pre>
</blockquote>
<p>On April 7th, 2013, 2:48 p.m. UTC, <b>Ashwin Rajeev</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 dont think that is a good idea. IMO either select this or stick to the old one.</pre>
</blockquote>
<p>On April 7th, 2013, 3:06 p.m. UTC, <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;">Why are you against something that offers more versatility and offers different ways of playing?</pre>
</blockquote>
<p>On April 9th, 2013, 4:12 p.m. UTC, <b>Anant Pushkar</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;">If we allow user to toggle between two modes, we must have two scoring algorithms as scoring in projectile mode is a little more difficult.</pre>
</blockquote>
<p>On April 9th, 2013, 10:39 p.m. UTC, <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;">Agreed.</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;">Ping?</pre>
<br />
<p>- Albert</p>
<br />
<p>On April 6th, 2013, 3:06 p.m. UTC, Anant Pushkar wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KDE Games.</div>
<div>By Anant Pushkar.</div>
<p style="color: grey;"><i>Updated April 6, 2013, 3:06 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;">The bombs earlier used to fall in straight line vertically. In this patch, I have made the path of the bomb a projectile just like in real life to make it look realistic. A side effect of this is that hitting the buildings in the beginning would be a little difficult. To tackle this, the average height of the first two buildings have been reduced.
Also, in cases when the plane crashes and not many blocks are still left, taking an aim from a larger height becomes difficult. Due to this the player has to wait till the plane itself comes down to a desired height. So, I have added a function that allows the player to lower the altitude of the plane by pressing the down button. I have a number of other features in mind that can be implemented as well. I will be soon submitting them.</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 would also like to add that I have tested the changes exhaustively and have found it to work well and improve the user experience.</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>src/board.h <span style="color: grey">(3816b95)</span></li>
<li>src/board.cpp <span style="color: grey">(1dd99a3)</span></li>
<li>src/bomb.h <span style="color: grey">(adf8541)</span></li>
<li>src/bomb.cpp <span style="color: grey">(7d361f5)</span></li>
<li>src/bomber.cpp <span style="color: grey">(4cd276f)</span></li>
<li>src/bomberwidget.h <span style="color: grey">(d807294)</span></li>
<li>src/bomberwidget.cpp <span style="color: grey">(497d8e1)</span></li>
<li>src/explodable.h <span style="color: grey">(aca7735)</span></li>
<li>src/explodable.cpp <span style="color: grey">(773f55f)</span></li>
<li>src/main.cpp <span style="color: grey">(05f4686)</span></li>
<li>src/plane.h <span style="color: grey">(acc8c8b)</span></li>
<li>src/plane.cpp <span style="color: grey">(a805415)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/109885/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>