<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/113528/">http://git.reviewboard.kde.org/r/113528/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On November 1st, 2013, 11:33 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;">That situation you describe looks weird, i should either not be able to end the turn or the "send action" should be canceled if i end the turn</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;">Now as you state that, it indeed looks weird. This however was introduced with the introduction of the end turn action. I updated the patch so the action state and the button state are kept synchronised.</pre>
<br />
<p>- Alexander</p>
<br />
<p>On November 1st, 2013, 3:24 a.m. UTC, Alexander Schuch 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 Alexander Schuch.</div>
<p style="color: grey;"><i>Updated Nov. 1, 2013, 3:24 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
konquest
</div>
<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 recently added KStandardGameAction::endTurn() action now uses the "default" shortcut of Alt+E. The action is added to the toolbar (and to the menu bar) by default. The custom "end turn" button is removed.
The custom "end turn" button is enabled and disabled based on context. So if the GUI expects the user to enter the number of ships to send from one planet to another, the button is disabled and does not work. This does not happen for the (new) action anymore. This basically is not a problem, except that the ship input field allowed range (m_shipValidator) is not updated.
Assume a planet has 10 ships and the user starts but does not finish the command sequence to send these ships to another planet by not (yet) entering the number of ships. If the user ends the turn at this point, the planet gets its production of ships and then has 20 ships (assuming 10 production). The accepted input range is not updated and the user cannot enter a value larger than 10. The user of course can abort the command sequence and start over to fix this problem.
I think this is not a big problem. If you believe it is, I will of course try to fix it.</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>gameview.h <span style="color: grey">(47b9463)</span></li>
<li>gameview.cc <span style="color: grey">(a01bfde)</span></li>
<li>konquestui.rc <span style="color: grey">(cd43d33)</span></li>
<li>mainwin.h <span style="color: grey">(d72fd46)</span></li>
<li>mainwin.cc <span style="color: grey">(929c61d)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/113528/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>