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

Prevent placing towers too close #76

Merged
merged 8 commits into from
Feb 26, 2022

Conversation

ddabble
Copy link
Member

@ddabble ddabble commented Feb 18, 2022

Depends on #75; this PR should change base to develop automatically after #75 has been merged.

This prevents players from placing towers on neighboring cells (1-cell radius) of other towers and the player base. The tower construction hologram is also colored red when "hovering" over a cell it's prevented from being placed on.

It's now not offset -0.53 units on the Z-axis anymore, and it's been rotated 30 degrees, so that the player base's hexagonal plinth is aligned with
the hex cell grid.
...as there is no need for setting it from elsewhere.
This reduces the performance cost.
The axe, lightning and wheel turret holo prefabs are now prefab variants of `TowerHolo_BasePrefab.prefab`.
@ddabble ddabble mentioned this pull request Feb 19, 2022
Base automatically changed from fix/cleanup-after-merge to develop February 24, 2022 22:49
…acing-towers-too-close

# Conflicts:
#	Assets/Prefabs/Towers/AxeTurret/AxeTurretHolo.prefab
#	Assets/Prefabs/Towers/LightningTurret/LightningTurretHolo.prefab
#	Assets/Prefabs/Towers/WheelTurret/WheelTurretHolo.prefab
@ddabble ddabble marked this pull request as ready for review February 24, 2022 23:09
Copy link
Contributor

@ThomasJGabrielsen ThomasJGabrielsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As far as i tested it works well!

@ddabble ddabble merged commit c745461 into develop Feb 26, 2022
@ddabble ddabble deleted the feature/prevent-placing-towers-too-close branch February 26, 2022 15:14
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.

2 participants