Opened 7 years ago

Closed 7 years ago

Last modified 4 years ago

#1853 closed enhancement (fixed)

[PATCH] Foundations should require 100% health for construction to be completed

Reported by: Jonathan Waller Owned by: wraitii
Priority: Must Have Milestone: Alpha 13
Component: UI & Simulation Keywords: simple, patch
Cc: Patch:

Description

See the forum thread http://www.wildfiregames.com/forum/index.php?showtopic=16977 for more details.

Attachments (1)

Foundation.patch (4.5 KB) - added by Manuel Núñez 7 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 7 years ago by Kieran P

Milestone: BacklogAlpha 13

comment:2 Changed 7 years ago by Kieran P

Type: taskenhancement

Changed 7 years ago by Manuel Núñez

Attachment: Foundation.patch added

comment:3 Changed 7 years ago by Manuel Núñez

Keywords: review patch added
Summary: Foundations should require 100% health for construction to be completed[PATCH] Foundations should require 100% health for construction to be completed

comment:4 Changed 7 years ago by Manuel Núñez

The patch also makes the building preview animation consistent with the current building percentage, sinking it if necessary.

comment:5 Changed 7 years ago by wraitii

Is the "sinking" actually wanted?

comment:6 Changed 7 years ago by leper

Yes. (Already cleared that up on IRC, but I'm adding it here to so that nobody does some unneeded work)

comment:7 Changed 7 years ago by wraitii

Patch seems fine. Commit?

comment:8 Changed 7 years ago by wraitii

Owner: set to wraitii
Resolution: fixed
Status: newclosed

In 13263:

Foundations now require 100% health for construction to be completed. Progress now linked with health. Patch by Malcom. Fixes #1853

comment:9 Changed 4 years ago by sanderd17

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