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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Target bones - Seraphim Tech 2 Land #4083

Merged
merged 6 commits into from
Aug 4, 2022

Conversation

Garanas
Copy link
Member

@Garanas Garanas commented Jul 31, 2022

image
Old and new collision box

image
Target location

Note that this unit has no target bones set, even though it looks like it is set to the turret 馃槃

@Garanas
Copy link
Member Author

Garanas commented Jul 31, 2022

image
Old and new collision box

image
Target location

Target bone is set to the Torso, which doesn't move that move in practice.

@Garanas
Copy link
Member Author

Garanas commented Jul 31, 2022

image
Old and new collision box

image
Target location

Also fixes the hover effect being behind the engineer, instead of being below it. The engineer has no target bones, due to the (negative) offset weapons aim slightly above the center of the engineer.

@Garanas
Copy link
Member Author

Garanas commented Jul 31, 2022

image
Old and new collision box

image
Target location

No target bones are set, the beam offsets slightly due to the negative offset. In practice this is negligible.

@Garanas
Copy link
Member Author

Garanas commented Jul 31, 2022

image
Old and new collision box

image
Target location

No target bones are set, there is a small offset vertically causing the beam to aim slightly lower than usual.

@Garanas
Copy link
Member Author

Garanas commented Jul 31, 2022

image

Fixed some of the effects used on the hover unit, adapted its elevation slightly.

@Garanas Garanas marked this pull request as ready for review July 31, 2022 07:40
@Garanas
Copy link
Member Author

Garanas commented Jul 31, 2022

This pull request is ready for a review, merging it toggles the task Tech 2 Land units for the Seraphim faction of #4067

@Garanas Garanas mentioned this pull request Jul 31, 2022
84 tasks
@Garanas
Copy link
Member Author

Garanas commented Jul 31, 2022

here-is-mp4-11.mp4

Test setup of this pull request - it looks surprisingly peaceful.

@Garanas Garanas merged commit bc9be13 into deploy/fafdevelop Aug 4, 2022
@Garanas Garanas deleted the tb/seraphim-tech2-land branch August 22, 2022 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant