-
Notifications
You must be signed in to change notification settings - Fork 19
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
Fix: Nuke Cannon Shell Detonation Effect Has Offset #1122
Conversation
bc34389
to
0cb3110
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
This change does not work correctly when targeting unit instead of terrain. Same for both weapons. generals.2022-09-06.12-59-35-71.mp4 |
0cb3110
to
ca2c14d
Compare
Good catch. Dumb game. Should work now. Note to self: Don't test nuke explosion effects on Nuke Battlemasters. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good now.
Good to merge? |
Spawns effect via dummy, which should not suffer impressions.