Design comparison
Solution retrospective
No questions on this one - Going back to basics for my first attempt with React.
Of course feedback is always appreciated!
Cheers
Ollie
Community feedback
- @denieldenPosted over 2 years ago
Hi Ollie, I took some time to look at your solution and you did a great job!
Also I add
main
tag and wrap the card for Accessibility and remove all unnecessary code, the less you write the better as well as being clearer: for example thediv
container of image. Overall you did well πHope this help and happy coding!
Marked as helpful1@ohermans1Posted over 2 years agoThanks for the great feedback @denielden! I still have a bit of learning to do around basic tidy code in React - So I will certainly take your feedback into account on my next project!
1 - @buneeIsSloPosted over 2 years ago
Whoa! I began learning React just yesterday. Your solution looks super clean and responds well too! Great job Ollie :)
Marked as helpful1@ohermans1Posted over 2 years ago@buneeIsSlo How are you learning! I'm working through a Udemy course, which I'm finding really good. Its called "React - The complete guide"
1@buneeIsSloPosted over 2 years ago@ohermans1 Well, at the moment I've been watching the Full Modern React Tutorial Playlist from "The Net Ninja". Once I'm done with this playlist, I'll crank out a few simple projects, and if things start going over my head, I might invest in a course on Udemy.
Marked as helpful1
Please log in to post a comment
Log in with GitHubJoin 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