Opened 8 years ago

Last modified 8 years ago

#4033 closed defect

[PATCH] Siege Engines are not shown in the summary — at Version 4

Reported by: Imarok Owned by: Mate-86
Priority: Should Have Milestone: Alpha 21
Component: UI & Simulation Keywords: simple patch
Cc: Patch:

Description (last modified by elexis)

  1. Siege Engines should be listed in the summary
  2. The captured statistic for Siege Engines should be displayed/created too

http://trac.wildfiregames.com/raw-attachment/ticket/4033/captured-units.png

http://trac.wildfiregames.com/raw-attachment/ticket/4033/captured-building.png

http://trac.wildfiregames.com/raw-attachment/ticket/4033/military-score.png

Change History (9)

comment:1 by Mate-86, 8 years ago

Owner: set to Mate-86
Status: new → assigned

by Mate-86, 8 years ago

Attachment: captured-units.png​ added

by Mate-86, 8 years ago

Attachment: captured-building.png​ added

by Mate-86, 8 years ago

Attachment: military-score.png​ added

comment:2 by Mate-86, 8 years ago

feedback from bb_:

  • captureable summary for Siege units and Total only => fixed
  • horizontal ordering of unit types could be changed to "... siege, naval, trader" => fixed
  • need to double check values printed (captured are marked as lost for the other player) => fixed
  • units and building headers need to be in the same order => fixed
Last edited 8 years ago by Mate-86 (previous) (diff)

comment:3 by Mate-86, 8 years ago

Keywords: rfc patch added
Summary: Siege Engines are not shown in the summary → [PATCH] Siege Engines are not shown in the summary

comment:4 by elexis, 8 years ago

Description: modified (diff)
Milestone: Backlog → Alpha 21

by Mate-86, 8 years ago

Attachment: siege-units-summary.patch​ added

v2

by Mate-86, 8 years ago

Attachment: units-summary-fixed.png​ added
Note: See TracTickets for help on using tickets.