MantisBT - VCMI
View Issue Details
0000819VCMIGUI - Hero screen / Exchange windowpublic2011-09-20 13:242014-05-30 17:40
Zamolxis 
Tow 
normaltrivialalways
closedfixed 
0.86 
0.87 
0000819: ESC does not close the Hero screen
Initially I wanted to report that ESC does not close neither the Hero screen, nor the Hero Exchange screen, but I checked in OH3 and saw that in the meeting screen ESC did nothing. Perhaps NWC coded it as such because pressing ESC by mistake in the Exchange screen would cost you movement points if you want the two heroes to 'meet' again.
No tags attached.
patch CHeroWindow.patch (862) 2011-09-20 17:21
https://bugs.vcmi.eu/file_download.php?file_id=812&type=bug
Issue History
2011-09-20 13:24ZamolxisNew Issue
2011-09-20 13:35ZamolxisNote Added: 0002031
2011-09-20 17:21GeshFile Added: CHeroWindow.patch
2011-09-20 17:22GeshNote Added: 0002039
2011-09-22 14:24TowNote Added: 0002041
2011-09-22 14:24TowStatusnew => resolved
2011-09-22 14:24TowFixed in Version => 0.89
2011-09-22 14:24TowResolutionopen => fixed
2011-09-22 14:24TowAssigned To => Tow
2014-05-30 17:40beegeeStatusresolved => closed

Notes
(0002031)
Zamolxis   
2011-09-20 13:35   
Checking the OH3 behavior reminded me of a frustrating thing: the (maybe) thousands of times I closed the Hero Exchange screen, to realize only after I forgot to exchange one creature or one artifact. Of course having the heroes meet again was one solution, but it came with the cost of movement points, which in Multiplayer or Impossible Single Player can have a notable impact.

This is pretty off-topic, but I would just like to raise the 'food for thought' suggestion to discuss if removing the movement cost for Hero meeting would be a gameplay change which would be embraced or bashed by the players. Because if everyone hated that, we could maybe change it, and then ESC to close the Exchange window as well would be a welcomed improvement for easy intuitive playing (at least my default key where my hand jumps to close any windows is ESC).
(0002039)
Gesh   
2011-09-20 17:22   
Hi, I just uploaded a patch for you. Ive dropped one in TOW's mailbox. It should fix that.
(0002041)
Tow   
2011-09-22 14:24   
Patch has been committed in r2388, thanks for contribution!