-
Notifications
You must be signed in to change notification settings - Fork 12
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
"Create New Microgame" option in menu #62
Conversation
🎉 Please test your changes using this link! ✨👉 https://polite-glacier-037563110.2.azurestaticapps.net 👈✨ |
🎉 Please test your changes using this link! ✨👉 https://polite-glacier-037563110.2.azurestaticapps.net 👈✨ |
🎉 Please test your changes using this link! ✨👉 https://polite-glacier-037563110.2.azurestaticapps.net 👈✨ |
🎉 Please test your changes using this link! ✨👉 https://polite-glacier-037563110.2.azurestaticapps.net 👈✨ |
🎉 Please test your changes using this link! ✨👉 https://polite-glacier-037563110.2.azurestaticapps.net 👈✨ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only left a tiny nitpick! Looks good to me!
|
||
enum InputType { | ||
MASH, # TODO - i don't love this for arrows+button but it seems to be the (WarioWare) Gold standard | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I usually have the last type of an enum be "SIZE" and have found that it's quite a useful bookend in an enum.
- You could query how many input types you have (InputType.SIZE)
- It feels like you're restricting the enum on one end - you can add things before SIZE, but not after it. Which is helpful when enums show in editor dropdowns because reordering an enum messes up what a dropdown is set to.
No description provided.