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

Display dynamic text objects instead of baking text into assets #4

Merged
merged 6 commits into from Dec 21, 2016

Conversation

stefanlesser
Copy link

This pull request removes text that was baked into assets and displays dynamic text objects instead. This will be helpful when translating the game into other languages.
I'm not proud of the repetitive code, and for sure this could be refactored into much leaner code, but I've never used PIXI before, and this does the trick. ;-)

Note: The Post_Post_Credits scene with the social buttons still uses text in assets. This is a bit trickier to change than it was in all other cases.

Some assets are also no longer necessary (in the Quote and Credits scenes), because removing the text made them empty, transparent images. I've still kept the images, and the references in the code to them. It would probably be good to remove the assets completely.

@stefanlesser
Copy link
Author

Oh, and my work of editing the assets is pretty crappy, too… the preload background in particular. Sorry, only had Preview to do it.

@ncase
Copy link
Owner

ncase commented Dec 7, 2016

Thank you again for doing all this! I'm currently finishing up my next interactive project (it's about alternative voting systems) but once I've finished it (by next week), I'll pull all this, and have a German version of WBWWB live somewhere! (As well as a new repo for others who want to help translate it) Many thanks again!

@terales
Copy link

terales commented Dec 7, 2016

@ncase can you announce new repository in this one or on your blog? I want to translate it into Russian.

@ncase
Copy link
Owner

ncase commented Dec 8, 2016

@terales: I'll announce it here on this repo! (and also on my Twitter) Thank you!

@ncase ncase merged commit 15c0142 into ncase:master Dec 21, 2016
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.

None yet

3 participants