Opened 12 years ago
Closed 12 years ago
#1489 closed defect (fixed)
[PATCH] Add UI command/hotkey to override enemy gather with attack
| Reported by: | Peter | Owned by: | ben |
|---|---|---|---|
| Priority: | Should Have | Milestone: | Alpha 12 |
| Component: | UI & Simulation | Keywords: | |
| Cc: | Priqvel@… | Patch: |
Description
When I clicked on field of enemy, my soldiers start harvesting, I think they were supposed to attack it =) I guess u need to change it
Attachments (1)
Change History (8)
comment:1 by , 12 years ago
| Component: | Core engine → UI & Simulation |
|---|---|
| Keywords: | Harvest enemy field removed |
| Milestone: | Backlog → Alpha 11 |
| Priority: | Must Have → Should Have |
| Summary: | cursor function → Can harvest from enemy fields instead of destroy them |
comment:2 by , 12 years ago
| Keywords: | simple added |
|---|---|
| Milestone: | Alpha 11 → Backlog |
| Summary: | Can harvest from enemy fields instead of destroy them → Add UI command/hotkey to override enemy gather with attack |
comment:3 by , 12 years ago
Another example where switching order type could be useful: once we have capturing, if a unit can both capture and attack a target, it might be nice to switch between them.
by , 12 years ago
| Attachment: | Hotkey added.patch added |
|---|
comment:4 by , 12 years ago
| Keywords: | review added |
|---|---|
| Summary: | Add UI command/hotkey to override enemy gather with attack → [PATCH] Add UI command/hotkey to override enemy gather with attack |
Added CTRL as attack hotkey
comment:5 by , 12 years ago
| Milestone: | Backlog → Alpha 12 |
|---|
comment:6 by , 12 years ago
| Keywords: | simple review removed |
|---|
Thanks for the patch. I think it's the right idea, but there is a conflict with the Ctrl hotkey already being used for garrisoning -- your new code is never run. I made a minor change of swapping the hotkey order and moving the attack check, that seems to fix it.
Note:
See TracTickets
for help on using tickets.
Gathering enemy "resources" was specifically enabled after discussion (it's more realistic and provides additional benefit for conquering enemy lands). Previously enemy fields couldn't be gathered. I think all we lack is the UI control to choose between attacking and gathering enemy resources. Gathering has priority over attack in the UI so you never get a choice.