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

CSGPolygon Add interpolation (Scaling/Moving) in mode Path #41499

Closed
wants to merge 12 commits into from

Conversation

stebulba
Copy link
Contributor

@stebulba stebulba commented Aug 25, 2020

This new feature from my proposal:
Add Scaling/Modify a CSG Polygon in PathFollow when PathNode is drawing the mesh
godotengine/godot-proposals#1062

  • No conflict with anything else.
  • No performance drop and no difference when is not active.
  • path_joined is working with a manipulation of the path_curve.

I did't compile from the master. I had issues to compile Godot 4. But on 3.2.3.rc1 is running perfectly.

The interpolation work using a Curve who interpolate between two polygon array. If both polygons are equal, a single scaling mode is active.

If the path_polygon2 is empty, when you active the interpolation bool, you make a copy of the first polygon to the second. Is a way to reset or help you for editing the polygons.

Enjoy !

@Calinou Calinou added cherrypick:3.x Considered for cherry-picking into a future 3.x release enhancement topic:core labels Aug 25, 2020
@Calinou Calinou added this to the 4.0 milestone Aug 25, 2020
@stebulba
Copy link
Contributor Author

stebulba commented Jul 5, 2021

I am re-writing the code to be compatible with the change #49313
90 % done and I will add more functionality / compatibility for all modes.

@stebulba
Copy link
Contributor Author

stebulba commented Sep 29, 2021

Supersede by #53195
The cherrypick label can be remove.

@akien-mga akien-mga closed this Sep 29, 2021
@akien-mga akien-mga added archived and removed cherrypick:3.x Considered for cherry-picking into a future 3.x release labels Sep 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants