Opened 14 months ago
Last modified 11 months ago
#6764 new enhancement
Burning units lack visualization of fire
Reported by: | Alex Theodoulidis | Owned by: | marder |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 27 |
Component: | UI & Simulation | Keywords: | iberian, fireship, fire, animation, sound |
Cc: | Alex Theodoulidis | Patch: | Phab:D4973 |
Description (last modified by )
E.g. When an Iberian fireship attacks an enemy ship, there is neither animation nor sound of the enemy ship being burned.
Attachments (1)
Change History (10)
by , 14 months ago
Attachment: | IberianFlagship#6764.mp4 added |
---|
comment:1 by , 14 months ago
Component: | Simulation → Art & Animation |
---|---|
Description: | modified (diff) |
Keywords: | fireship added; flagship removed |
Milestone: | Alpha 28 → Alpha 27 |
Patch: | Alpha 26 |
Priority: | Must Have → Should Have |
Summary: | Iberian Flagship attack totally lacks simulation → Iberian fireship lacks attack animation |
comment:2 by , 14 months ago
Hmm not sure it's art and animation. AFAIK we don't support variants for status effects.
comment:3 by , 14 months ago
I was more thinking about little fire particles that just "attack" the other ship / fly away from the fireship in all directions. It is right now indeed not possible to visualize the status effect.
comment:4 by , 13 months ago
Status effects not being supported is a big obstacle. This is why I set Alpha 28 as Milestone. Right now, a sound of wood being burned and some fire particles as @marder suggested are better than nothing.
At a later time, we could maybe implement the fireship in a more realistic and creative way, because it truly has potential as a unique unit. I actually have some ideas in mind.
comment:5 by , 13 months ago
Component: | Art & Animation → UI & Simulation |
---|---|
Description: | modified (diff) |
Keywords: | fire added; ship collision removed |
Milestone: | Alpha 27 → Alpha 28 |
Owner: | set to |
Patch: | → Phab:D4973 |
Summary: | Iberian fireship lacks attack animation → Burning units lack visualization of fire |
Type: | defect → enhancement |
comment:7 by , 13 months ago
rewording to focus on the fire issue. The ship overlapping deserves a different ticket (possibly reopen #5084 -> although at the moment the overlap is a wontfix, since it is a known tradeoff to make the boat movement at least somewhat less annoying).
comment:8 by , 13 months ago
Replying to Stan`:
Is this a dupe of #131 ?
Mhh I would say no, since this only focuses on status effects, while #131 is about damage in general, but you can argue otherwise.
That being said, something like the proposed patch could be adapted to work with damage in general. Could possible have some performance implication tho (when spawning new units on attack/ damage taken, being captured).
Thanks for the report (The ship overlapping is indeed a more general problem)