Submitted about 2 years ago
Intro section with dropdown navigation
@mdabdulrahman
Design comparison
SolutionDesign
Solution retrospective
I did my best 😊
Community feedback
- @JaneMorozPosted about 2 years ago
Hey! You solution is great 💪
The only thing I've noticed straight away:
- You need to wrap the main content of the page into the <main> tag. It will solve all this landmark issues.
- Images need
alt
attribute. You can usealt=""
if the image is only for decoration. You can read more here.
Keep it up! And good luck 🍀
Marked as helpful1@mdabdulrahmanPosted about 2 years ago@JaneMoroz Thanks for your guidance! I will change it
1
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