-
Notifications
You must be signed in to change notification settings - Fork 39
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
Labyrinth #17
base: main
Are you sure you want to change the base?
Labyrinth #17
Conversation
…ld and connected it to the fetch-logic. Also implemented conditional statements so that if a username is not typed in, you get a message that need to type in a username to start
…nd fixed a focusRef for the input-field
Added positioning on direction-buttons using a dynamic classname.
…e to click enter to submit the username
…, also fixed so we show error-message if we have error & that the loading-message is a h2
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.
Very nice UX/UI 😊 I'm missing the descriptions to help guide the user, but will approve because you fulfilled the requirements.
actions: [], | ||
coordinates: null, | ||
description: null, | ||
gameMode: false, |
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.
What does this mean? Could it be named hasStarted or isGameOn or something, if it has to do with that?
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.
Well structured Zustand store 🐻
Netlify link
https://labyrinth-adventure-game.netlify.app/
Collaborators
[ericamechler, FridaMari]