-
Notifications
You must be signed in to change notification settings - Fork 50
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
feat: Change the Hierarchy to support egui::CollapsingHeader
#121
Conversation
…own that is also interactable.
Added a check for each entity to see if the children that it has implement the |
Thanks for the pull request! |
Looks almost like a working solution already. I only found a bug that when adding a child to an entity, the parent entity is moved one space along with the child. Only the child should be moved one space. After that, the pull request will be ready for merging. |
I hope you don't mind, I fixed the described problem and got rid of code duplication. If you like my solution, pull request is ready to be merged with the main branch and will be in 0.3 version. |
No worries on the edits and all, that was what I was looking for! Otherwise it looks all good to me. |
This is a quick change that converts the
selectable_label
s being used for the hierarchy toCollapsingHeader
s.There is still an issue with some entities that are brought in from
.glb
files it seems and I would love some feedback about that.Otherwise I believe that this is just a small feature update.