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

updated pepperoni icon to pizza emoji and updated launch screen #8

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

Conversation

erikbatista42
Copy link
Contributor

This was one of the issues and I thought that I could fix it, so I did. Instead of an image of a pepperoni, I changed it to a pizza emoji. You can also see I have modified some variable names in the code for better readability and updated the launch screen for the design to make more sense.

@fulldecent
Copy link
Owner

This is very cool, thank you.

And I was thinking to have Mr. Pizza eat all kinds of emoji coming onto the screen.

But the issue is that this messes up all the alliteration in the app's astounding annotations (i.e. App Store description). Any idea what to do with that?

@fulldecent
Copy link
Owner

Please add screenshots to review here. Also it seems pepperoni.png is not needed if this is merged.

@erikbatista42
Copy link
Contributor Author

erikbatista42 commented Mar 22, 2021

Hey! You're welcome :)

If the purpose of Mr.Pizza is to eat pizza (like the title of the game - Pizza Man), can I ask, why we would want to have him eat all kinds of emojis? Wouldn't that change the purpose of the game?

And for sure, I'll soon make sure to add screenshots and remove pepperoni.png - I thought I removed this file!

@fulldecent
Copy link
Owner

Ah. The name is because the hero (at center) is a pizza. The things that he is eating are pepperoni.

It wouldn't make sense for Mr. Pizza to be eating more pizzas, that cannibalism!

But if he is eating things that are typically pizza toppings, then that might be appropriate. There are probably lots of emojis for those potential toppings. And it might even be cool to have the variety of toppings change with each level (every ten points).

And again, a critical consideration here is that the intro text (App Store metadata) has to be updated.

@fulldecent
Copy link
Owner

I can see this stuff is relevant to the project scope // updating at https://github.com/fulldecent/pizzaman/blob/master/README.md

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.

2 participants