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

Changed Aircraft Husk explosion #14106

Merged
merged 1 commit into from Oct 14, 2017

Conversation

Projects
None yet
6 participants
@abc013
Contributor

abc013 commented Oct 3, 2017

Before

airplaneexplosion1

After

airplaneexplosion2

In my opinion this looks much better.

@GraionDilach

This comment has been minimized.

Show comment
Hide comment
@GraionDilach

GraionDilach Oct 3, 2017

Contributor

How does this fare with buildable airplanes?

Contributor

GraionDilach commented Oct 3, 2017

How does this fare with buildable airplanes?

@abc013

This comment has been minimized.

Show comment
Hide comment
@abc013

abc013 Oct 3, 2017

Contributor

This change also applies to buildable aircraft.
airplaneexplosion3

Contributor

abc013 commented Oct 3, 2017

This change also applies to buildable aircraft.
airplaneexplosion3

@GraionDilach

This comment has been minimized.

Show comment
Hide comment
@GraionDilach

GraionDilach Oct 3, 2017

Contributor

Yes, I noticed and I believe that the explosion is too big for them. Your proposal fits well to the paraplane, but I'd go with napalm2 on helis for certain.

Contributor

GraionDilach commented Oct 3, 2017

Yes, I noticed and I believe that the explosion is too big for them. Your proposal fits well to the paraplane, but I'd go with napalm2 on helis for certain.

@abc013

This comment has been minimized.

Show comment
Hide comment
@abc013

abc013 Oct 3, 2017

Contributor

Updated:
airplaneexplosion4

Contributor

abc013 commented Oct 3, 2017

Updated:
airplaneexplosion4

@Smittytron

The Badger crash looks good.

I tested the heli crash on top of infantry, and napalm2 looks a little out of scale with the infantry deaths around it. I honestly think the current explosion doesn't look bad with the smaller helicopters if you think large napalm is too much.

@pchote

pchote approved these changes Oct 14, 2017

@pchote pchote added the PR: Needs +2 label Oct 14, 2017

@ltem

ltem approved these changes Oct 14, 2017

@reaperrr reaperrr merged commit c8b2a7d into OpenRA:bleed Oct 14, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@reaperrr

This comment has been minimized.

Show comment
Hide comment
@reaperrr
Contributor

reaperrr commented Oct 14, 2017

@abc013 abc013 deleted the abc013:aircraftexplodepolish branch Oct 14, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment