<div dir="ltr"><div dir="ltr"><div dir="ltr"><div>+kde-games-devel</div><div><br></div></div></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Feb 10, 2019 at 12:32 AM Glenn Pavlovic <<a href="mailto:gpavman@hughes.net">gpavman@hughes.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello,<br>
I made some (what I consider to be improvements) changes to ksirk<br>
4.14.3. Patch file attached (generated with LC_ALL=C TZ=UTC0 diff -Naur<br>
ksirk-4.14.3 ksirk-4.14.3.revised > ksirk-4.14.3.patch)<br>
<br></blockquote><div><div><br></div><div>Hi Glenn,</div><div><br></div><div>Thank you for your interest and contribution to ksirk.<br></div><div>Unfortunately,
 I couldn't try your patch due to various reasons and troubles in 
compiling locally 4.14 branch. Master and recent branches are all based 
on KF5 and Qt5 for some time now.</div><div><br></div><div><div>As you noticed, some improvements can be made to get UI out of the way for some game plays.<br></div><div><br></div></div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
1. Added a button to the invaisonSlider dialog to set the slider to the<br>
maximum.<br></blockquote><div>Is it possible to modify slider to match your needs instead of adding new button. QSlider is used here and current default page step is 10. You can move pages by leftclicking on slider or with PageUp/Down. End key will set the slider to the maximum, while Home would set it to the minimum.</div><div>If all of these is not enough, or can't be used to satisfy what you are trying to accomplish, maybe an option could be added to always set maximum value for invasion slider instead of minimum?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
2. Added a slider to the right dialog to allow multiple armies to be<br>
added toi a country at once.<br></blockquote><div>If I understand correctly, once the country has been selected, the slider would determine how many armies would be placed, instead of single click - one army?</div><div>I agree this might be useful especially later in the game with large number of armies.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
3. Added a dialog (accesed via CTRL/SHIFT/META+right click) to allow<br>
multiple armies to be extracted from a country at once, if there are 5<br>
or more available to be extracted.<br></blockquote><div>Not sure I understand where and how this new dialog is used.</div><div><br></div><div><br></div><div>If it is possible, please try to build latest code (master) and send patches via phabricator. That and keeping kde-games-devel in the loop might get you faster responses. <br></div><div><br></div><div><br></div><div><div dir="ltr"><div dir="ltr"><div>Best regards,</div><div>Nemanja<br></div></div></div> </div></div></div>