-
-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
Node.raise() is misleading #28948
Comments
It is called |
Definitely a candidate for renaming, neither the docs nor the name say what it does. |
The name is also confusing to anyone expecting |
If this method is desired, I think it would make sense to move it to CanvasItem. |
@aaronfranke the problem with doing this is that while |
@bojidar-bg does it have a use case in 3D though? It seems to be way less common. |
Tree order affects e.g. processing order, so technically it has use in any node. |
@KoBeWi This has an effect for sure, but then the use case is different. |
The doc was updated, but what should we do with the method? Rename? (could be called |
Godot version:
3.1.1
Issue description:
The description of this method says:
What does the "top of the array" mean? I interpreted it as it moves it as first child, but actually it's the opposite. The documentation should clearly state where the node will move instead of using vague terms.
The text was updated successfully, but these errors were encountered: