Design comparison
Solution retrospective
This is my first challenge as a beginner and I would love your feedback. Done in 4 hours.
Community feedback
- @PhoenixDev22Posted almost 3 years ago
Greeting MELLANIE , Congratulation on completing your First Frontend mentor project, your solution looks nice. I think you have done a great work for your first challenge.
• Wrap everything in your body in <main> ... OR use semantic tags ... OR give role="" to the direct children of your <body> ... Click here to read more
• Add alternate text to the images. Setting the
alt
text for the images in<div class="container">
to an empty string. That's because the purpose of those images are mostly decorative and need not be announced by assistive technology like screen readers.Excellent solution, really well done! I really hope this feedback helps. Happy coding
Marked as helpful1 - Account deleted
Pretty good approach! Very similar. Try to use less padding/margin rules and keep practicing!
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