Skip to content

Display arrows from factory spawn to target #489

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 7 commits into from
Jun 20, 2023

Conversation

0HyperCube
Copy link
Contributor

@0HyperCube 0HyperCube commented Jun 12, 2023

Closes #434.

First draft implemented in PR #484.

Follow ups:

  • Update Add Terrain Heightmap Loading #2: mention that the lines must be updated to work properly on non-flat surface.
  • Add functionality which hides the line when the camera is too far away to avoid clutter.

Copy link
Collaborator

@Indy2222 Indy2222 left a comment

Choose a reason for hiding this comment

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

We are half way there, but the PR still requires some care.

@0HyperCube 0HyperCube requested a review from Indy2222 June 20, 2023 18:21
Indy2222 added 5 commits June 20, 2023 23:12
This will be faster in cases with many lines because it does not iterate over all of them. Furthermore, this changes despawn() to despawn_recursive() so it is more future proof and consistent with the other despawn in the module.
@Indy2222
Copy link
Collaborator

Udeps pipeline is failing due to rust-lang/rust#112831

@Indy2222 Indy2222 merged commit 0dd9804 into DigitalExtinction:main Jun 20, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Display "arrows" from selected factories to delivery location
2 participants