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

allows different waypoints #335

Open
wants to merge 1 commit into
base: Preview4_0
Choose a base branch
from

Conversation

lukaspj
Copy link
Contributor

@lukaspj lukaspj commented Oct 3, 2020

I wanted to share this for some time, not sure if this is the best way to do it, but before it falls into the void :) this will allows to have at the same time different waypoint/markers, add more "WayPointMarker"....

Well you could always add from script, but this PR allows to add it from the World Editor.

See an example, at the right the green old shape. Now you can have different shapes and colors used for enemies, allies, objects.... I can do other PR with the markers I use, it have different shapes and colors, if required.
29991497-96c010aa-8f4d-11e7-9d28-f5b5f08c36a6
29991651-8de9e282-8f50-11e7-91a2-0d2836df54c7

Original PR from the GarageGames repo:
GarageGames/Torque3D#2076

@lukaspj lukaspj added QoL Enhancement Non functional enhancements that make work easier (New or improved UI parts for example) Script Issue as part of template/module script labels Oct 3, 2020
@lukaspj
Copy link
Contributor Author

lukaspj commented Oct 3, 2020

The changes seems fairly straight-forward. I don't have time to properly test this, but can't seem the harm in it.

@JeffProgrammer
Copy link
Member

The changes seem to not be doing anything. It looks like equivalent code...also if you wanted to add more, you'd have to change the conditions now instead of just adding a new switch statement. Does this actually fix a bug?

@lukaspj
Copy link
Contributor Author

lukaspj commented Oct 25, 2020

This is a QoL not a bug fix.

It seemingly insignificant, but right now it's a switch that defaults to a SpawnSphere unless it's set to one of two specific datablocks.
With this change, you can have different Waypoint Datablocks without hitting the default of spawnsphere

@lukaspj
Copy link
Contributor Author

lukaspj commented Oct 25, 2020

Actually this could also be accomplished with just changing the switch statement afaict.
The idea is to allow arbitrary waypoint datablocks anyways

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QoL Enhancement Non functional enhancements that make work easier (New or improved UI parts for example) Script Issue as part of template/module script
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants