Ticket #521 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

Rally Points for Buildings

Reported by: evans Owned by: evans
Priority: Nice to Have Milestone:
Component: Core engine Keywords: review
Cc: philip

Description

The ability to create Rally Points for buildings by right clicking on the ground after selecting a building. All newly created units from that building will walk towards the rally point soon after creation.

Attachments

RallyPoints.diff​ (7.8 KB) - added by evans 3 years ago.
RallyPoint.js​ (404 bytes) - added by evans 3 years ago.
binaries/data/mods/public/simulation/components/RallyPoint.js
RallyPoint.2.js​ (39 bytes) - added by evans 3 years ago.
binaries/data/mods/public/simulation/components/interfaces/RallyPoint.js

Change History

Changed 3 years ago by evans

Changed 3 years ago by evans

binaries/data/mods/public/simulation/components/RallyPoint.js

Changed 3 years ago by evans

binaries/data/mods/public/simulation/components/interfaces/RallyPoint.js

comment:1 Changed 3 years ago by evans

  • Cc philip added
  • Keywords review added
  • Status changed from new to assigned

comment:2 Changed 3 years ago by Philip

Thanks! This seems basically fine, though I ended up making a few minor modifications:

  • Added right-click-on-selected-building command to unset the rally point.
  • Changed where the GUI updates the rally point display, so that it updates when you destroy a selected building.
  • Rearranged the rally point rendering code to make #524 easier.
  • Fixed removal of old rally point entities so they vanish instantly (DestroyEntity has a delay).
  • Prevented displaying rally points for enemy buildings.

comment:3 Changed 3 years ago by philip

  • Status changed from assigned to closed
  • Resolution set to fixed

(In [7849]) # Add rally points for newly-trained units, based on patch from evans. Fixes #521.

comment:4 Changed 3 years ago by anonymous

  • Milestone Unclassified deleted

Milestone Unclassified deleted

Note: See TracTickets for help on using tickets.