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

Added Cell and Wold position to TerrainGeometryOverlay #16442

Merged
merged 1 commit into from May 21, 2019

Conversation

@teinarss
Copy link
Contributor

commented Apr 23, 2019

Use case: Got tired on counting cells and this would help when developing the pathfinder and such things.

@teinarss teinarss force-pushed the teinarss:terrain_overlay branch 3 times, most recently from d249ea7 to 052c918 Apr 27, 2019

@teinarss teinarss marked this pull request as ready for review Apr 30, 2019

@abcdefg30
Copy link
Member

left a comment

Needs a rebase.

OpenRA.Mods.Common/Widgets/Logic/Ingame/DebugLogic.cs Outdated Show resolved Hide resolved

@teinarss teinarss force-pushed the teinarss:terrain_overlay branch from 052c918 to 6edb85c May 18, 2019

@teinarss

This comment has been minimized.

Copy link
Contributor Author

commented May 18, 2019

Rebased and added the missing header

@reaperrr

This comment has been minimized.

Copy link
Contributor

commented May 19, 2019

👍 after that.

@teinarss teinarss force-pushed the teinarss:terrain_overlay branch from 6edb85c to 6255dcc May 19, 2019

@teinarss

This comment has been minimized.

Copy link
Contributor Author

commented May 19, 2019

fixed!

@teinarss teinarss force-pushed the teinarss:terrain_overlay branch from 6255dcc to 3ee2457 May 20, 2019

@teinarss teinarss force-pushed the teinarss:terrain_overlay branch from 3ee2457 to 69a659d May 21, 2019

@teinarss

This comment has been minimized.

Copy link
Contributor Author

commented May 21, 2019

Updated

@pchote
pchote approved these changes May 21, 2019

@pchote pchote merged commit ef47a33 into OpenRA:bleed May 21, 2019

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.