-
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
Alma and Wen - Labyrinth project #3
base: main
Are you sure you want to change the base?
Conversation
Movement logic
Style start page
Add some background images and make restart button work to change background
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.
Nice and clean UI and well structured code too. Good job!
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.
Structured store and well named states and actions, well done ⭐
set({ | ||
coordinates: data.coordinates, | ||
description: data.description, | ||
actions: data.actions, |
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.
Would be nice if you also saved the description of the actions, that's quite a big part of the game 👀
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.
Love this font!
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.
Nice loading animation you found!
<> | ||
<FirstPage /> | ||
</> |
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.
No need for fragments when you're only returning one element
error: null, | ||
loggedIn: false, | ||
startGame: async (name) => { | ||
const encryptedName = name + new Date() |
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.
Hehe, nifty!
Netlify link
https://alma-wen-labyrinth.netlify.app/
Collaborators
[almaherris]