Skip to content
This repository has been archived by the owner on Sep 27, 2024. It is now read-only.

Fix drawing layer organisation #54

Open
Cullen-Enerdyne opened this issue Nov 10, 2021 · 6 comments
Open

Fix drawing layer organisation #54

Cullen-Enerdyne opened this issue Nov 10, 2021 · 6 comments
Assignees
Labels
Bug Something isn't working or is broken. Feature Request Request to add a feature to the application.

Comments

@Cullen-Enerdyne
Copy link
Collaborator

Make sure entities are being drawn at the correct layer.

@Cullen-Enerdyne Cullen-Enerdyne added Bug Something isn't working or is broken. Feature Request Request to add a feature to the application. labels Nov 10, 2021
@Cullen-Enerdyne Cullen-Enerdyne self-assigned this Nov 10, 2021
@AlexvZyl
Copy link
Owner

AlexvZyl commented Nov 10, 2021

Might be a good idea to make some sort of array that layers the data from top to bottom according to an index, instead of manually entering numbers.

@Cullen-Enerdyne Cullen-Enerdyne changed the title Fix drawign layer organisation Fix drawing layer organisation Nov 13, 2021
@Cullen-Enerdyne
Copy link
Collaborator Author

I like this idea, @AlexEnerdyne. Can we discuss this? Do you have an idea on how to implement this?

@Cullen-Enerdyne
Copy link
Collaborator Author

@AlexEnerdyne , this is currently being handled by manually keeping track of which entities are in which layer, and hard-coding this. Any ideas on how to do this properly (and efficiently) would be great. I can think of ways of populating vectors of entities, then layering them in the order in which they appear, however the implementation I'm thinking of would require updating the layer of all entities whenever another layer is added. This obviously isn't viable.

@AlexvZyl
Copy link
Owner

@Cullen-Enerdyne Correct me if I am wrong, but the only reason we need so many layers is because of the way blending currently works?

@Cullen-Enerdyne
Copy link
Collaborator Author

@Cullen-Enerdyne Correct me if I am wrong, but the only reason we need so many layers is because of the way blending currently works?

That's part of the reason, but sometimes we also need it for occlusion.

@AlexvZyl
Copy link
Owner

AlexvZyl commented Dec 1, 2021

This is not a full solution, but we could definitely add Move back and Move forward options to the context menu. Then all components are added to the same layer, and the user can adjust as needed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug Something isn't working or is broken. Feature Request Request to add a feature to the application.
Projects
None yet
Development

No branches or pull requests

2 participants