Ticket #1206 (closed enhancement: fixed)
[PATCH] Garrison UI Enhancement: Shift-Click Removal
| Reported by: | michael | Owned by: | leper |
|---|---|---|---|
| Priority: | Nice to Have | Milestone: | Alpha 10 |
| Component: | UI & Simulation | Keywords: | Garrison, UI |
| Cc: |
Description
Shift-Click the unit icons in Garrison UI panel to unload all of that type.
Be sure to make this distinction known in the tooltip: "Single-click to unload 1. Shift-click to unload all of this type."
Attachments
Change History
comment:1 Changed 14 months ago by leper
- Keywords UI, review added; UI removed
- Summary changed from Garrison UI Enhancement: Shift-Click Removal to [PATCH] Garrison UI Enhancement: Shift-Click Removal
comment:3 Changed 14 months ago by quantumstate
A few comments on the patch:
You reuse the batchtrain hotkey. I think it would be better to define a new one since they are different tasks.
You send the "Unable to ungarrison unit" from within a loop. It should be taken out of the loop so that only one message is sent. It would be nice if you could make the message plural/singular as well.
The AI scripts use the unload command (the common-api has a function, it isn't actually used yet though), so this needs updating to match your changes. Generally if you ever change a function or command it is good to do a search so you can see all of the places it is used.
comment:4 Changed 14 months ago by leper
- Owner set to leper
Nice to know that multiple hotkeys can use the same key. Thanks for reminding me to grep through the ai folders.
