Design comparison
Solution retrospective
Open to constructive criticism.
Community feedback
- @danielmrz-devPosted 9 months ago
Hello @Vinzz34!
Your solution looks excelent!
I have just one suggestion:
š Consider this tweak for your HTML code:
To improve semantic clarity, opt for
<h1>
over<div>
for your main title.Think of
<div>
and<span>
in HTML like plain boxes or placeholders. They're handy for holding content, but they don't tell us anything about what's inside or what it's meant for on the webpage.It's more than just text size ā it's about structuring your content effectively:
<h1>
to<h6>
are used to define HTML headings, with<h1>
being the most significant.- Stick to one
<h1>
per page for the main title, and maintain the titles hierarchy with<h1>
,<h2>
<h3>
, and so on.
While these adjustments might not alter the visual appearance much, they significantly enhance semantic clarity, SEO optimization, and accessibility.
Hope this suggestion proves helpful! Keep up the great work!
Marked as helpful1@Vinzz34Posted 9 months agoThanks for the feedback @danielmrz-dev
I'll definitely follow this in my upcoming projects
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