[Kde-games-devel] Review Request 113528: Remove custom "end turn" button.

Alexander Schuch aschuch247 at gmail.com
Fri Nov 1 03:24:06 UTC 2013


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

Review request for KDE Games.


Repository: konquest


Description
-------

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.


Diffs
-----

  gameview.h 47b9463 
  gameview.cc a01bfde 
  konquestui.rc cd43d33 
  mainwin.h d72fd46 
  mainwin.cc 929c61d 

Diff: http://git.reviewboard.kde.org/r/113528/diff/


Testing
-------


Thanks,

Alexander Schuch

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


More information about the kde-games-devel mailing list