P
Andrés R.W.• 420
@amporabipo
Posted
You have to pay much more attention to the sizes, don't worry. I am in the same
Marked as helpful
0
Design comparison
Community feedback
Daniel 🛸• 44,230
@danielmrz-dev
Posted
Hello Joseph!
Your solution looks great!
I have a suggestion for improvement:
- For semantic reasons, use
mainto wrap the main content instead of adiv. This tag change does not impact your project visually and makes your HTML code more semantic, improving SEO optimization as well as the accessibility of your project.
I hope it helps!
Other than that, great job!
0
Please log in to post a comment
Log in with GitHub
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