Design comparison
Solution retrospective
Your feedback is invaluable to me. I sincerely appreciate your response.
Community feedback
- @danielmrz-devPosted 8 months ago
Hey @jayaxsurya!
Your solution is really impressive!
I've got a couple of ideas (about how to use HTML better) that could make it even stronger:
š First: Think about using
<main>
to wrap your main content instead of<div>
.Imagine
<div>
and<span>
in HTML as basic containers. They're good for holding stuff, but they don't tell us much about what's inside or its purpose on the webpage.š Second: Don't skip heading levels ā start with
<h1>
, then use<h2>
, and so on..It's more than just text size ā it's about structuring your content effectively:
- The
<h1>
to<h6>
tags are used to define HTML headings. <h1>
is for the most important heading.<h6>
is for the least important heading.- Stick to just one
<h1>
per page ā it should be the main title for the whole page.
These tweaks might not change how your page looks, but they'll make your HTML code clearer and help with SEO and accessibility.
Hope that's helpful!
Keep up the great work!
1@jayaxsuryaPosted 8 months ago@danielmrz-dev,
Thanks a lot for taking the time to review my work and for your valuable suggestions! I really appreciate your insights.
Your tips are incredibly helpful, and I'll make sure to implement them in my future projects.
Thanks again for sharing your expertise!
0 - The
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