-
-
Notifications
You must be signed in to change notification settings - Fork 485
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
Group every selected set of layers for its artboard independently #1992
base: master
Are you sure you want to change the base?
Conversation
I think this would have several issues. If the artboard was not selected, then it would not be added to the hashmap and nothing would happen. The best way to approach this would be to collect all the layer paths, and check if there is an artboard in the path. If so, then the ROOT_PARENT gets popped from the path. If the path is just one item at this point, it can be filtered out (since it is an artboard). Once these paths are collected, a hash map can be used to group the paths based on the last element. Then each group of paths can be grouped. The parent is the deepest common ancestor, and the rest of the logic can be applied from there. |
Okay I'll rework it |
a4eccdd
to
b18537e
Compare
@adamgerhant can you review? |
I pushed my changes/fixes about 2 weeks ago, and it was ready to merge after that. |
@adamgerhant in that situation, could you please remember to leave an approval on the PR? Thanks :) |
I'm confused by what exactly happened with the commit history. It seems my commit got edited and then forced pushed? I reviewed/tested it though and its ready to merge |
I checked this and I think it's okay |
No description provided.