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

Add water and terrain as child game object #265

Merged
merged 9 commits into from
Nov 18, 2023

Conversation

Dgzt
Copy link
Collaborator

@Dgzt Dgzt commented Oct 31, 2023

This PR contains some fixes:

  1. If we move the mouse to a game object in Outline and click to Add water or Add terrain then it will be added as child game object.

  2. If we create a water at root in Outline and move it into a game object as a child game object then it will be rendered.

  3. There is the findComponentsByType(Array<Component> out, Component.Type type, boolean includeChilds) method in GameObject class and it's javadoc says that if the includeChilds variable is true then it searches in children too, but in this case this method doesn't search in the current game object, only in children. So with this fixes if recursive variable is true then it will search in the current game object and recursively too. Added this changes into runtime's CHANGES file as braking changes.

  4. Rendering water in runtime if water is child object.

@Dgzt Dgzt added the bug Something isn't working label Oct 31, 2023
@Dgzt Dgzt changed the title Add water and terrain as child Add water and terrain as child game object Oct 31, 2023
@Dgzt Dgzt marked this pull request as ready for review October 31, 2023 15:32
# Conflicts:
#	commons/src/main/com/mbrlabs/mundus/commons/scene3d/GameObject.java
#	editor/src/main/com/mbrlabs/mundus/editor/ui/modules/outline/Outline.kt
#	gdx-runtime/CHANGES
Copy link
Owner

@JamesTKhan JamesTKhan left a comment

Choose a reason for hiding this comment

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

Looks good to me, I can merge once conflict is resolved.

@Dgzt
Copy link
Collaborator Author

Dgzt commented Nov 16, 2023

Looks good to me, I can merge once conflict is resolved.

Done.

@JamesTKhan JamesTKhan merged commit a022064 into JamesTKhan:master Nov 18, 2023
@Dgzt Dgzt deleted the add-water-and-terrain-as-child branch November 26, 2023 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants