Anonymous | Login | 2024-12-03 06:02 UTC |
My View | View Issues | Change Log | Roadmap |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0000216 | VCMI | GUI - Town screen | public | 2009-11-07 11:21 | 2013-06-02 14:38 | ||||
Reporter | Zamolxis | ||||||||
Assigned To | Tow | ||||||||
Priority | high | Severity | trivial | Reproducibility | always | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | 0.74 | ||||||||
Target Version | Fixed in Version | 0.89 | |||||||
Summary | 0000216: VCMI-H3 discrepancy: VCMI uses hotkey 'T' to open Marketplace, while H3 used it for Towns on the map & Tight in hero screen | ||||||||
Description | H3 used 'T' for Towns on Adventure Map (select/focus first town in the list, then cycle through list) and for Tight formation in Hero Screen. So 'T' needs to be re-allocated as per H3, and we need to find another hotkey for the Marketplace. We cannot use 'M', because it is taken by Move. The only letters not used on H3 Adventure map as hotkeys were: B, F, G, J, R, X & Y. Of these, I'd say our best options are: B = Buy or R = Resources Whichever you choose, it's maybe good to implement it also in the CTRL+/SHIFT+ combinations, same as NWC did and to be consistent with how also the other hotkeys were implemented. I don't have a preference between B or R. The only thing is that R would conflict with Replay in the Adventure Options screen (and we could code it for Recruit in some other screens), so with the Marketplace coded to be called by hotkey from any screen, maybe it's best to go for B, which is not used in any H3 interface (and can't think of other use we can give it). | ||||||||
Additional Information | If for any reason you prefer not using B or R, the next hotkeys which intuitively might pop to mind, are CTRL+M or CTRL+T. The problem is that in H3 they had the same function as M and T, however they were never documented in the H3 manual, so we are not so bound to implement them identically. And same goes for SHIFT+ combinations. If we do implement CTRL+/SHIFT+ combinations exactly like H3 (alternatives to single hotkeys), then I guess we are left only with ALT+M or ALT+T for the Marketplace window (perhaps both?). These were not used in H3. However the risk here is that they may be less intuitive (though we will have a manual to take care of that I guess). | ||||||||
Tags | No-harm enhancement | ||||||||
Attached Files | |||||||||
Relationships | |||||||||||
|
Notes | |
(0001445) Zamolxis (viewer) 2010-11-22 11:53 edited on: 2012-03-11 13:23 |
Thanks to phomm (0000602) we identified another discrepancy: in H3, T triggered Tight formation in Hero screen, while in VCMI opens Marketplace as well. So as part of this fix, 'T' should also be re-allocated to Tight formation in Hero screen (updated in Report description as well). |
(0002304) Zamolxis (viewer) 2012-03-11 13:27 edited on: 2012-03-11 13:27 |
Priority increased because - moving between H3 & VCMI for tests - I keep hitting T wanting to go to Towns, but I get something else in VCMI. I think this should change soon, before VCMI players get used to the wrong hotkey. |
(0002334) Tow (developer) 2012-04-08 01:18 |
Fixed in r2613. Hotkey for marketplace will be CTRL+T. Implemented original T behavior for town switching. Tight formation hotkey was already implemented, the effect probably was suppresed by marketplace window. |
(0003639) Zamolxis (viewer) 2013-06-02 14:38 |
Verified in 0.93 and confirmed as fixed. Actually meanwhile hotkeys changed to following: - On Adv.map "T" is for towns and "CTRL+T" for Trading Post - both "T" and "CTRL+T" only select Tight formation in Hero screen Maybe ideally CTRL+T should have opened trading post in Hero screen also, for rare situations like Gelu needing money to upgrade Sharpshooters, but that's such a rare need, that I don't think somebody will really miss it. |
Issue History | |||
Date Modified | Username | Field | Change |
2009-11-07 11:21 | Zamolxis | New Issue | |
2009-11-07 12:11 | Zamolxis | Summary | 'T' was implemented to open Marketplace window, however in H3 'T' was used for cycling through the towns in the list => 'T' was implemented to open Marketplace window, however in H3 'T' was used for select & cycle through the towns in the list |
2009-11-07 12:11 | Zamolxis | Description Updated | View Revisions |
2009-11-07 12:11 | Zamolxis | Description Updated | View Revisions |
2009-11-28 07:55 | Warmonger | Category | Other => GUI - Town screen |
2010-11-22 11:16 | Zamolxis | Summary | 'T' was implemented to open Marketplace window, however in H3 'T' was used for select & cycle through the towns in the list => VCMI-H3 discrepancy: VCMI uses hotkey 'T' to open Marketplace, while H3 used it for towns (select & cycle through list) |
2010-11-22 11:16 | Zamolxis | Description Updated | View Revisions |
2010-11-22 11:20 | Zamolxis | Description Updated | View Revisions |
2010-11-22 11:20 | Zamolxis | Relationship added | related to 0000602 |
2010-11-22 11:24 | Zamolxis | Description Updated | View Revisions |
2010-11-22 11:53 | Zamolxis | Note Added: 0001445 | |
2010-11-22 11:55 | Zamolxis | Severity | feature => trivial |
2010-11-22 12:51 | Zamolxis | Tag Attached: No-harm enhancement | |
2012-03-01 13:58 | Zamolxis | Description Updated | View Revisions |
2012-03-11 13:21 | Zamolxis | Summary | VCMI-H3 discrepancy: VCMI uses hotkey 'T' to open Marketplace, while H3 used it for towns (select & cycle through list) => VCMI-H3 discrepancy: VCMI uses hotkey 'T' to open Marketplace, while H3 used it for Towns on the map & Tight in hero screen |
2012-03-11 13:21 | Zamolxis | Description Updated | View Revisions |
2012-03-11 13:21 | Zamolxis | Additional Information Updated | View Revisions |
2012-03-11 13:23 | Zamolxis | Note Edited: 0001445 | View Revisions |
2012-03-11 13:27 | Zamolxis | Note Added: 0002304 | |
2012-03-11 13:27 | Zamolxis | Priority | normal => high |
2012-03-11 13:27 | Zamolxis | Note Edited: 0002304 | View Revisions |
2012-03-11 13:30 | Zamolxis | Description Updated | View Revisions |
2012-03-11 15:22 | Zamolxis | Relationship added | related to 0000907 |
2012-04-08 01:18 | Tow | Note Added: 0002334 | |
2012-04-08 01:18 | Tow | Status | new => resolved |
2012-04-08 01:18 | Tow | Fixed in Version | => 0.89 |
2012-04-08 01:18 | Tow | Resolution | open => fixed |
2012-04-08 01:18 | Tow | Assigned To | => Tow |
2013-06-02 14:38 | Zamolxis | Note Added: 0003639 | |
2013-06-02 14:38 | Zamolxis | Status | resolved => closed |
Copyright © 2000 - 2024 MantisBT Team |