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

Adding monkey 🐵 #24

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Adding monkey 🐵 #24

wants to merge 6 commits into from

Conversation

CRYPTICNOTME
Copy link

adding my monkey with fixes

Copy link

vercel bot commented Jan 24, 2025

@CRYPTICNOTME is attempting to deploy a commit to the Hack Club Team on Vercel.

A member of the Team first needs to authorize it.

@acornitum acornitum added the needs review needs acon to check over it , ,,,, label Jan 29, 2025
@acornitum
Copy link
Contributor

Hey - this is great!

Just a few more things -

  • Would it be possible to polish up the background a little more? Something that would make it look better is to either make the black lines that outline the items less thick/inconsistent, or to not have those lines at all. See also: lineless art! Most pixel art backgrounds I see these days are lineless. What a lot of people do for pixel art is also show depth in the background via brightness of the color - for example, having colors into the front lighter, and for every layer behind, making it darker, or vise versa. Here's a good example of that!
  • Would it be possible for you to add a start screen?

Thanks so much for submitting! I can't wait to see it when those changes have been made.

Note: don't open a new PR, just commit changes to this current thread + send a comment when changes have been made! This response took so long because I ran into so many git(?) errors trying to open this game up but it's fixed now :"D

@acornitum acornitum removed the needs review needs acon to check over it , ,,,, label Jan 30, 2025
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