MantisBT - VCMI
View Issue Details
0001406VCMIGUI - Hero screen / Exchange windowpublic2013-08-12 18:522016-11-11 08:26
Zamolxis 
AVS 
lowtrivialhave not tried
closedfixed 
0.93 
0.98d 
0001406: We can select ally troops in the exchange window
As per screenshot:
- we can select ally troops (and more than just 1 stack)
- when at least one is selected, also the Split Army button becomes available
- if we click on Split Army, all other slots which can serve as destination get selected

However in spite of the above, we cannot "steal"/move/split ally creatures, even if slot selection might indicate so (which basically makes this a GUI bug only so far).

In H3 we can never select an ally creature slot (hence Split Army also never becomes available). The only time ally slots get selected, is when we select one of our troops and - if the case - if we click on Split with one of our troops selected.

Interestingly enough though, H3 was a bit buggy here: even though also ally slots were getting the yellow frame (empty + same troop slots), we could not use them - we could only split within our army. Trying to split into the ally army was just deselecting everything. I guess we have two ways to better implement this:
a) Either the Split Army button becomes enabled only on our hero's end, and slots get framed/available only in his army.
b) Or same H3 graphical behavior, but then also functionality should follow:
- we should be able to split one of our stacks into an empty slot (at least)
- and ideally we should be able to split also into a same-troop ally slot, but without the possibility of stealing; for example by having the slider already at the extreme left when the window opens, and the original size should be considered by VCMI as the minimum
I used a modified version of the VCMI_Tests map, in which I put Red & Green in the same team from MapEd then had Ivor meet Uland next to him.
No tags attached.
jpg 2013-08-12_AllyExchange.jpg (80,610) 2013-08-12 18:52
https://bugs.vcmi.eu/file_download.php?file_id=1418&type=bug
jpg
Issue History
2013-08-12 18:52ZamolxisNew Issue
2013-08-12 18:52ZamolxisFile Added: 2013-08-12_AllyExchange.jpg
2013-08-12 18:54ZamolxisSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=2371#r2371
2013-08-12 18:55ZamolxisDescription Updatedbug_revision_view_page.php?rev_id=2373#r2373
2013-08-12 18:55ZamolxisSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=2374#r2374
2015-09-14 16:51AVSNote Added: 0005857
2015-09-14 16:51AVSStatusnew => resolved
2015-09-14 16:51AVSFixed in Version => 0.98d
2015-09-14 16:51AVSResolutionopen => fixed
2015-09-14 16:51AVSAssigned To => AVS
2016-11-11 08:26SXXStatusresolved => closed

Notes
(0005857)
AVS   
2015-09-14 16:51   
was been fixed in https://github.com/vcmi/vcmi/pull/115 [^]