Opened 15 years ago

Closed 12 years ago

Last modified 8 years ago

#277 closed enhancement (fixed)

[PATCH]Unit selections

Reported by: Zervox Owned by: gerbilOFdoom
Priority: Nice to Have Milestone: Alpha 9
Component: Core engine Keywords:
Cc: Patch:

Description

-When dragging the selection box over screen, only select military units if there are any, if not, then make it only choose villagers, and only let 1 building get selected at once instead of(lets say) 10, currently if you drag you get villagers,buildings and military units in one selection

Attachments (1)

#277.patch (2.4 KB ) - added by gerbilOFdoom 12 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 by Michael D. Hafer, 15 years ago

I would suggest that click-drag not select any buildings at all. Buildings should only be selected by direct click and also through double-click (selecting all buildings of the same type). I also agree with the drag box preferring to select military units first. Perhaps we can have left-click-drag select only support units (females, trader, healers) and left+right-click-drag select only military units. We need to discuss this further.

comment:2 by Zervox, 15 years ago

Yes, that would actually be better, except for the left+right part, it would be an annoying starting issue about left+right click, as its not used in other games. Just my 2 cents.

comment:3 by (none), 14 years ago

Milestone: Open Source Release

Milestone Open Source Release deleted

comment:4 by Andrew, 14 years ago

Milestone: Backlog

comment:5 by gerbilOFdoom, 12 years ago

Owner: set to gerbilOFdoom
Status: newassigned

I'll work on right-click bandboxing to select only military units.

comment:6 by gerbilOFdoom, 12 years ago

Keywords: review added
Milestone: BacklogAlpha 9
Summary: Unit selections[PATCH]Unit selections

Solution: Add hotkey for unit selection (alt) to select only preferred & non-support units.

by gerbilOFdoom, 12 years ago

Attachment: #277.patch added

comment:7 by Jonathan Waller, 12 years ago

This looks good, I will commit it now. One minor coding style point is you should use spaces for alignment in the default.cfg file, otherwise people with different tab sizes will get misaligned comments.

comment:8 by Jonathan Waller, 12 years ago

Resolution: fixed
Status: assignedclosed

(In [10873]) Add military only drag select (Alt modifier) by gerbilOFdoom. Closes #277

comment:9 by sanderd17, 8 years ago

Keywords: review removed
Note: See TracTickets for help on using tickets.