Refactor screen state swapping#10180
Conversation
|
Also I was wondering if we maybe should call this 'Scene' instead of 'Screen', so we get TitleScene, GameScene, LoadingScene etc. |
|
This sound very similar to the (ambitious) goals the initial GameState PR #5979 set out with. We never got quite that far. I haven't looked at the code in detail yet, but good to see this resurface.
|
tjwrona
left a comment
There was a problem hiding this comment.
I don't officially know enough about the program structure at this point to feel comfortable approving/disapproving this review, but I've made a couple comments.
tjwrona
left a comment
There was a problem hiding this comment.
Just curious, but I don't have a problem with it. Why change the name from "screen" to "scene"? Is it because scene is a more generic term and a scene may not necessarily correspond to a screen?
duncanspumpkin
left a comment
There was a problem hiding this comment.
Looks good. I've not played with it myself.
|
This needs a fresh start |
Some initial work towards having multiple screen states. This will allow to have at some point loading screens that will do all the things in the background and simply swap the screen state after it has loaded all the required things, its still pretty raw and I would like to get some feedback like if we want to go this route.