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

Split the SceneGraph from the "state graph", and make the RenderLayer a property of Renderables #9

Open
1 task
Tomcc opened this issue May 3, 2015 · 0 comments

Comments

@Tomcc
Copy link
Owner

Tomcc commented May 3, 2015

Currently a GameState is also an object.
This is not ok, a GameState should be a loose collection of objects that doesn't depend on the structure of the Scene Graph, usually.
Eg. a child of the Viewport should be allowed to be in a different GameState than the viewport itself, or the VP should belong to both states.

Also, manually adding Renderable to queues is error prone and not flexible enough, so it would be better to build the layers by traversing the scenegraph when something changes.

After the refactoring:

  • remove all uses of setChildrenVisible
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

No branches or pull requests

1 participant