Opened 8 years ago

Last modified 3 years ago

#4253 new enhancement

[PATCH] Patrol modifier hotkey or option

Reported by: elexis Owned by:
Priority: Nice to Have Milestone: Backlog
Component: UI – In-game Keywords: patch, simple
Cc: Patch:

Description

For the patrol feature (#1720), there could be either a hotkey (mostly relevant after #2604 since nearly all keys are mapped already), or an options entry (similar to the batch train size option). The option would allow to chose between two options:

A) attacking all enemy units while patroling B) attacking all enemy units and structures while patroling

(We also thought about involving the unit stance somewhere (for example if the units are set to passive, they might not attack anything while patroling. Not convinced.)

Attachments (1)

patrol_prepare_v1.patch (2.8 KB ) - added by elexis 8 years ago.
Removes the duplicate "Unit" entry and makes it accessible from the GUI. Not sure if it's 100% safe (i.e. if the current order always exsists and if its always the patrol one), but it ought to.

Download all attachments as: .zip

Change History (8)

by elexis, 8 years ago

Attachment: patrol_prepare_v1.patch added

Removes the duplicate "Unit" entry and makes it accessible from the GUI. Not sure if it's 100% safe (i.e. if the current order always exsists and if its always the patrol one), but it ought to.

comment:1 by elexis, 8 years ago

Keywords: patch rfc added
Milestone: BacklogAlpha 22
Summary: Patrol modifier hotkey or option[PATCH] Patrol modifier hotkey or option

The cleanup patch could be committed beforehand, so that we catch possible issues in weird situations asap.

comment:2 by elexis, 7 years ago

In 18983:

Unify duplicate patrol target types hardcodings. Reviewed by fatherbushido long ago, refs #4253.

comment:3 by elexis, 7 years ago

Keywords: rfc removed

comment:4 by elexis, 7 years ago

Milestone: Alpha 22Backlog

Backlogging due to lack of progress

comment:5 by Imarok, 5 years ago

Component: UI & SimulationIn-game UI

Move tickets to In-game UI as UI & Simulation got some sub components.

comment:6 by Silier, 3 years ago

Keywords: simple removed
severity: simple

comment:7 by Silier, 3 years ago

Keywords: simple added
Note: See TracTickets for help on using tickets.