-
Notifications
You must be signed in to change notification settings - Fork 44
feat: Added layer sorting option #67
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
Conversation
|
Code looks nice! I'll give it some more review and pull it in when I get to it 🙂 |
|
Thank you! I just realized that I forgot to delete a Debug.log() in LDtkSortingOrder.cs (Apologize). |
|
I looked more into how the values decrement, and I think that I made a flaw in my observations. I assumed that the sorting order would decrement more than once per layer, because it used to do that in the far past. |
|
Check out this branch i pushed to. it works off of your own commit: Give me feedback, and if it seems good, i'll merge it in! |
|
Haven't checked the code, but this feature seems super useful! Does this sorting also include sorting layer or just the sorting number? |
That's a good question; it only affects the number, but it might be convenient if the sorting layer can also be specified. The conflict, though, is that intgrid tile assets can have a sorting layer set also, so one would have to take precedence. @WMsans There are other ponderings related to this for a menu that lists all layers, like a material field, a sorting layer field, etc. It would be great, but it would be a UI challenge to keep it clean |




Added layer sorting option
The user can customize the layer order through the main panel.
https://github.com/user-attachments/assets/7c999c31-eb16-473d-a19e-ba6579c577fd