Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor sea chicken improvements #3252

Closed
4 tasks
sprunk opened this issue Sep 25, 2018 · 1 comment
Closed
4 tasks

Minor sea chicken improvements #3252

sprunk opened this issue Sep 25, 2018 · 1 comment

Comments

@sprunk
Copy link
Member

sprunk commented Sep 25, 2018

Chickens were never designed for sea and ultimately these won't change that but they seem like a low mana way to improve the situation:

  • give chicken projectiles the ability to pierce water.
  • probably make melee chickens walk underwater (because the juicier eco buildings don't float)
  • #sonarLives
  • there are reports that chickens have some sort of slowdown on water, check whether this is true and remove.
@ArchShaman
Copy link
Contributor

ArchShaman commented Sep 26, 2018

I've got a branch that's working on the issue. Basically I've changed the following:

  • Melee units walk underwater. (Taken from the duck unitdef)
  • Goo and spores now pierce the water. Spores is a standardization because some spore types cannot pierce the water, but others can.
  • All chicken units now have sonar = sight.
  • Dodo now walks underwater. (It's a "melee" unit, right?)

@sprunk sprunk closed this as completed in e7d9f9b Oct 7, 2018
lhog pushed a commit to lhog/Zero-K that referenced this issue Feb 15, 2019
 * chickens now have sonar and can see underwater
 * chickens no longer have speed penalties on sea
 * melee chickens now move on the oceanfloor

Fixes ZeroK-RTS#3252
GoogleFrog pushed a commit to danfireman/Zero-K that referenced this issue Feb 26, 2019
 * chickens now have sonar and can see underwater
 * chickens no longer have speed penalties on sea
 * melee chickens now move on the oceanfloor

Fixes ZeroK-RTS#3252
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants