react / styled-component / redux / yup

Solution retrospective
No specific questions.
I will deal with HTML validation issues later. I am open to all feedbacks & criticisms, thanks.
Please log in to post a comment
Log in with GitHubCommunity feedback
- @tediko
Hello, Jay! 👋
Congrats on finishing GURU challenge! 🎉 Forgive me for speaking so late, but I had a few days off. Your solution is flawless! I really like the animations you added. They look very smooth and the whole UI is great.
- I don't think you need
aria-label
s in your header navigation links. It seems to me that they're self-descriptive. - It is different with the cart button tho. I'd add
aria-label
to button and keep my imagealt
text empty. Also change this text to something more descriptive - 'cart icon image' doesn't speak much.
Good luck with that, have fun coding! 💪
Marked as helpful - I don't think you need
- @Kakasi-Daniel
Nice!
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord