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

Tweak Cruiser and Destroyer turret offsets #14945

Merged
merged 1 commit into from Mar 21, 2018

Conversation

Projects
None yet
3 participants
@reaperrr
Copy link
Contributor

reaperrr commented Mar 18, 2018

Moved back turret of cruiser a bit more backwards and destroyer turret a bit forward.
Makes them look a bit more like in the original (and in case of the destroyer, videos).

OLD:
ra-ship-turrets-old

NEW:
ra-ship-turrets

Tweak Cruiser and Destroyer turret offsets
Moved back turret of cruiser a bit more backwards and destroyer turret a bit forward.
@pchote

This comment has been minimized.

Copy link
Member

pchote commented Mar 19, 2018

Do you have a before shot for comparison?

@reaperrr

This comment has been minimized.

Copy link
Contributor Author

reaperrr commented Mar 19, 2018

Added a screenshot from bleed for comparison. Looks like the cruisers' back turret could go back another pixel.

@pchote

pchote approved these changes Mar 21, 2018

Copy link
Member

pchote left a comment

LGTM

@pchote pchote added the PR: Needs +2 label Mar 21, 2018

@abcdefg30 abcdefg30 merged commit 40d7b41 into OpenRA:bleed Mar 21, 2018

2 checks passed

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

This comment has been minimized.

Copy link
Member

abcdefg30 commented Mar 21, 2018

@reaperrr reaperrr deleted the reaperrr:tweak-ship-turrets branch May 7, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.