Skip to content

add despawn_children #5649

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

Open
PhaestusFox opened this issue Aug 10, 2022 · 0 comments
Open

add despawn_children #5649

PhaestusFox opened this issue Aug 10, 2022 · 0 comments
Labels
A-ECS Entities, components, systems, and events C-Feature A new feature, making something new possible

Comments

@PhaestusFox
Copy link

What problem does this solve or what need does it fill?

currently, despawn_descendents is a recursive way of removing all children from an entity without removing the entity itself,
I don't know what use case it would have but the face there is a difference between despawn and despawn recursive why not at this choice to this approach too

What solution would you like?

either despawn_children() as a method to despawn children but non-recursively
or despawn_descendants_recursive() to do what despawn_descendents() does and change it to be non-recursive

What alternative(s) have you considered?

could add a root_children() and root_child(index) methods to allow children to be made rood nodes would also make it easier to fix #5584

@PhaestusFox PhaestusFox added C-Feature A new feature, making something new possible S-Needs-Triage This issue needs to be labelled labels Aug 10, 2022
@Weibye Weibye added A-ECS Entities, components, systems, and events and removed S-Needs-Triage This issue needs to be labelled labels Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ECS Entities, components, systems, and events C-Feature A new feature, making something new possible
Projects
None yet
Development

No branches or pull requests

4 participants