Opened 11 years ago

Closed 10 years ago

Last modified 10 years ago

#2093 closed enhancement (fixed)

[PATCH] Ranged attack spread should be varied by distance

Reported by: sanderd17 Owned by: wraitii
Priority: Should Have Milestone: Alpha 16
Component: UI & Simulation Keywords:
Cc: Patch:

Description

When ranged units or ballistas are close to their target, their projectile is sometimes ridiculously far off. While it looks plausible on further distances.

So I suggest making the spread variable by distance.

This patch used the spread defined in the attack component to be the spread on the maximum range, and makes the spread smaller for anything else.

Attachments (1)

distance_spread.diff (2.2 KB ) - added by sanderd17 11 years ago.

Download all attachments as: .zip

Change History (7)

by sanderd17, 11 years ago

Attachment: distance_spread.diff added

comment:1 by Jonathan Waller, 10 years ago

Keywords: design added; review removed

comment:2 by wraitii, 10 years ago

This seems to be a no-brainer good patch to me (as far as design goes).

Last edited 10 years ago by wraitii (previous) (diff)

comment:3 by leper, 10 years ago

Milestone: Alpha 15Alpha 16

comment:4 by wraitii, 10 years ago

Owner: set to wraitii
Resolution: fixed
Status: newclosed

In 14425:

Spread is now linearly tied to distance to target. Fixes #2093 .

comment:5 by wraitii, 10 years ago

Committed in a slightly different version this morning, so it can be tested and reverted if not satisfactory.

I've made the relation linear as this makes more sense and is faster anyway. I've also kept the velocity thingy already in the code now.

comment:6 by wraitii, 10 years ago

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