Skip to content

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