-
Notifications
You must be signed in to change notification settings - Fork 32
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
Achievement modal upgrade #17
Comments
Hi, |
Hi Victoria! I'm happy to provide basic support for reading the code. If email isn't essential however I would prefer communication via these issues to keep it professional, but if you need private communications for assignment purposes etc then let me know. I usually reply pretty quickly as you can probably see. |
Hi Steven! Thanks again and Best, |
Hi Steve, Best, |
Hello Steve! I am Victoria's partner, Dannie (Tianying). We have a quick question regarding the formatting of the code. Would it be okay for us to move the CSS and Javascript code in index.html into different files (a javascript file and a css file)? Thank you, |
Hi Dannie, Go for it! Steven |
Hey @dannie-zhang and @vicdjy! Someone just took out the original drunk-pieces game mode (Issue #14), so I've put in issue #19 of similar difficulty to replace it. I'm happy to keep patching things up but if you'd like to reserve some issues please a) comment that you're reserving the issue on the issue itself, and b) let me know so I can put up more issues for others to work on. Also, you'll need to pull those changes before you push. All the best! |
Upgrade the achievement modal so that it appears as a smaller modal on top of the screen that fades automatically.
In doing so, also update the n-slayer achievement so that kills where you are killed count towards total kills.
The text was updated successfully, but these errors were encountered: