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

zOrder Support #204

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

zOrder Support #204

wants to merge 1 commit into from

Conversation

SquareLi
Copy link

@SquareLi SquareLi commented Dec 6, 2013

Added zOrder support for entity.

@markknol
Copy link
Contributor

markknol commented Dec 6, 2013

I would call it just 'order', this has nothing to do with a z-axis right?

@SquareLi
Copy link
Author

SquareLi commented Dec 6, 2013

When two objects overlap, the object has higher z-order will be on the top.

@volkipp
Copy link
Contributor

volkipp commented Dec 6, 2013

I think 'order' makes sense to me as well, because an entity doesn't necessarily even have a sprite attached to it.

@npretto
Copy link

npretto commented Dec 6, 2013

I'd honestly prefer "zOrder" to just "order", if i wanted to display something on top of something else, i'd look for a "z-something" property first, order may look like something more generic.
But I'm not sure, looking at the code it seems that the order in which entities get updates is the same as they get rendered right? That is actually just a more generic "order" then

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants