addImageFile fails silently about phaser

[BorisKozo]

Ok, it was my fault. I didn't call loader.load();
But that doesn't explain why create wasn't called. Do I have to call loader.load() for the create method to run?

[photonstorm]

If there is an init function then the boot process waits for loader.load to complete and then calls create. If you don't need to load anything I would remove init entirely. I've been thinking of renaming init to 'load' to make this more obvious, but I think you would have been ok if you had only had some docs to read :)

[BorisKozo]

I am writing a simple Snake game and I am struggling since I don't understand the best practices of using the library. Is there some IRC channel I can ask questions at?

[photonstorm]

Actually thinking about it, create only doesn't get called if loader isn't started. I.e. you could have loads of code in init and so long as none of that adds anything to the loader then create should be called. There is no irc atm, sorry. The best (only?!) place is the forum and looking at as many of the tests as you can. 1.0 is fully documented I promise :)

[BorisKozo]

I agree that the coupling between init and loader.load() is a little strange (and definitely not clear). I would even dare to say spahgettish :)

[photonstorm]

It actually helps keep your code significantly cleaner, but I am tempted to rename it to load, or maybe preload, as that is what you use it for. Then it's more obvious to leave it out if you don't need to load anything.

[BorisKozo]

Why wouldn't the framework call loader.load() itself if it relies on that call for the correct flow to continue?
You can allow the user to cancel the call through a property (e.g. loader.cancelLoad) but still. If the loader.load() function isn't called then the application is stuck which is IMHO is very bad.

[photonstorm]

Because there are properties of the load function that I want devs to set themselves and can't be second guessed. It only relies on it if you've created an init method. If you don't need to load anything then it shouldn't exist.

[BorisKozo]

I understand your point but don't you think it would be more elegant to put loader.load() here: https://github.com/photonstorm/phaser/blob/master/Phaser/Game.ts#L230

And put the two parameters you can supply to loader.load as properties of the loader itself.

So if previously I would do:
loader.load(fileLoadCallback, loadCompleteCallback) in the init method

And in my suggestion:
loader.fileLoadCallback = fileLoadCallback;
loader.loadCompleteCallback = loadCompleteCallback;

As a user of your library I think it would be much clearer that way.
This is my last comment on this, I promise :)