Opened 7 years ago

Closed 7 years ago

#4328 closed defect (fixed)

[PATCH] Linear splash damage for bolt shooters

Reported by: elexis Owned by: Imarok
Priority: Nice to Have Milestone: Alpha 22
Component: UI & Simulation Keywords: patch
Cc: fatherbushido Patch:

Description

Bolt shooters should have linear splash damage, thus being able to hit multiple units with one shot, making the bolt shooter more realistic and useful to the player.

Attachments (1)

splashforboltshooter.diff (965 bytes ) - added by fatherbushido 7 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 by fatherbushido, 7 years ago

It makes sense and would be an example of use of that functionnality.

by fatherbushido, 7 years ago

Attachment: splashforboltshooter.diff added

comment:2 by fatherbushido, 7 years ago

Keywords: patch review added
Milestone: BacklogAlpha 22
Summary: Linear splash damage for bolt shooters[PATCH] Linear splash damage for bolt shooters

Added that. (it will help in finding new bugs too).

Last edited 7 years ago by fatherbushido (previous) (diff)

comment:3 by elexis, 7 years ago

The catapult deals 15 Pierce and 35 Crush damage. The proposal 70 Pierce and 5 Crush is strong in comparison and one-shots citizen soldiers and reduces the health of unwounded champions by a third. The linear damage affects all units in that vector in the given range I assume. 8 meters should be enough space for 3-4 units, so the balancing seems just about right with the current slow attack rate.

Always wondered about the packtime of bolt shooters, significantly reducing their agility. The harder it becomes to retreat the unit, the more delayed is it's forward use going to be.

For completeness, the tooltip could be updated to also display the shape, for example "%(Shape)s Splash Damage" as the title.

comment:4 by fatherbushido, 7 years ago

(For complete information, there is a decay with distance in damage.)

Oh yes, do you want to do the tooltip? :p You'll be faster than me at that.

(And agility and so on, yes. Doing those special stance in UnitAI and/or fixing #2802 / #4359 could be nice. :/)

comment:5 by elexis, 7 years ago

(Decay -> Ah, good.) (Tooltip -> can do, but there are many seats of fire.) (Agility -> Compare to rams who move very fast. If they would need 5? second unpack time, people wouldn't build rams until they have full military dominance. Instead we can see rams being trained right after reaching age 3, as they can be relatively quickly retreated to the fortress, which I find more entertaining than waiting even longer)

comment:6 by fatherbushido, 7 years ago

Keywords: review removed

comment:7 by elexis, 7 years ago

As mentioned, I agree with the diff, so one could commit it as reviewed by me and keep that ticket simple and open for the tooltip.

comment:9 by Imarok, 7 years ago

Keywords: simple removed
Owner: set to Imarok

comment:10 by Imarok, 7 years ago

Resolution: fixed
Status: newclosed

In 19229:

Splash Damage shape tooltip

Reviewed by: elexis
Differential Revision: https://code.wildfiregames.com/D142
Fixes #4328

Note: See TracTickets for help on using tickets.