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

Ninas Project Labyrinth #212

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Ninas Project Labyrinth #212

wants to merge 11 commits into from

Conversation

NinaWald
Copy link

No description provided.

Copy link

@EvaSjoholm EvaSjoholm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really good work Nina and nice with the sound effect at the end, everything working as it should and loved your animation + background image.
All I could give as input is creating more folders to keep the design-code in separate files so it get's a bit cleaned up, styled components like you did use.
Great work, be proud!

}

useEffect(() => {
const audio = new Audio(soundFile); // Create an Audio object with the sound file

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great input with the sound-effect!

}
}, [showEndPage]);

const ResetBtn = styled.button`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe create a separate file for your design and styled components to keep function-files a bit cleaner/easy to follow :) But just an input, nothing major!

export const Direction = ({ actions = [] }) => {
const dispatch = useDispatch();

useEffect(() => {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done with this code!

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