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

Almanzo the Anglerfish #90

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

Almanzo the Anglerfish #90

wants to merge 2 commits into from

Conversation

lll11233
Copy link

@lll11233 lll11233 commented Feb 3, 2025

No description provided.

Copy link

vercel bot commented Feb 3, 2025

@lll11233 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 Feb 3, 2025
@acornitum
Copy link
Contributor

I really like the starting screen you have here! The art of this game is also pretty neat, I like the colors :D

Some suggestions I want you to do before I merge this!

  • Currently, on the timer, the text overlaps each other - could you make it not so that the time is actually visible?
  • The score doesn't reset when the timer resets currently
  • Try to get rid of the flicking on the fish! Also, maybe make it turn the other way when it's moving the other way, so it doesn't appear as if it's moving backwards?
  • Add an end screen and give the player an option to restart!

Thanks so much for submitting! Can't wait to see where it's at after you've made those changes.

Note: don't open a new PR, just commit changes to this current thread + send a comment when changes have been made!

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