Opened 2 years ago

Closed 2 years ago

#6400 closed enhancement (fixed)

Move some information from cmpFormation to cmpIdentity.

Reported by: Freagarach Owned by: Freagarach
Priority: Nice to Have Milestone: Alpha 26
Component: Simulation Keywords:
Cc: Patch: ​Phab:D4476

Description

Currently, the cmpFormation contains the name and icon (and a tooltip) of a formation. This overlaps with the function of cmpIdentity. We could just enable the cmpIdentity for formations and store the name and icon there. We could then also add some history about the formations and their specific names.

Reason: Reduce duplication accross components.

Change History (3)

comment:1 by Freagarach, 2 years ago

Keywords: simple added

comment:2 by Freagarach, 2 years ago

Keywords: simple removed
Patch: → Phab:D4476

comment:3 by Freagarach, 2 years ago

Owner: set to Freagarach
Resolution: → fixed
Status: new → closed

In 26476:

Move FormationName and Icon from cmpFormation to cmpIdentity.

Reduces duplication across components.
Allows to provide specific names and history for formations.

Differential revision: ​https://code.wildfiregames.com/D4476
Comments by: @bb, @Stan
Fixes #6400

Note: See TracTickets for help on using tickets.