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

Command "Wrap to node" #4

Open
astrochili opened this issue Jul 24, 2024 · 5 comments
Open

Command "Wrap to node" #4

astrochili opened this issue Jul 24, 2024 · 5 comments
Assignees
Labels
enhancement New feature or request

Comments

@astrochili
Copy link

In the hierarchy tree, it would be convenient to select a list of nodes, right-click and select "Wrap to node", which will create a node and place all selected nodes in it without global position offset.

This feature allows to complicate the hierarchy when an intermediate node is needed.

@Insality
Copy link
Owner

Insality commented Jul 24, 2024

LIke this feature and got some tests ealier to "group" and "ungroup" nodes, but since most users do export from Defold files, it's no very useful until there is an import function

@Insality
Copy link
Owner

But I'm sure it will be easy to add ;)

@Insality Insality self-assigned this Jul 24, 2024
@Insality Insality added the enhancement New feature or request label Jul 24, 2024
@Insality
Copy link
Owner

Insality commented Aug 3, 2024

Add the most simple implementation, you can check in at https://github.com/Insality/panthera/releases/tag/editor.804

Hotkeys: CMD + G and Shift + CMD + G (or ctrl instead of CMD)

It just make a wrap for selected nodes or delete parent if ungroup. No any additional checks and can be weird if trying to group the nodes from different hierarchy, but should be okay at start

@Insality Insality closed this as completed Aug 3, 2024
@astrochili
Copy link
Author

Grouping works fine 👍 It also seems to be a good idea to add this command to the context menu.

Ungrouping works in a little bit unintuitive way, as selecting a node affects the node above it. It would be more intuitive to call this function by selecting the node you want to ungroup.

@Insality
Copy link
Owner

Insality commented Aug 3, 2024

Good notice! WIll check and update later

@Insality Insality reopened this Aug 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants