Design comparison
Solution retrospective
Any opinion is welcome...
Community feedback
- @CSE-KylePosted 9 months ago
nice work on the project!
on the text below "html and css foundations" use the
width:
property to change the length of the text horizontally andline-height:
to separate the text vertically as shown in the solution. Other than that you're on the right track!Marked as helpful0 - @danielmrz-devPosted 9 months ago
Hello @LukebaKueno!
Your solution looks excelent!
I have just one suggestion:
š To improve semantic clarity, stick to one
<h1>
per page for the main title.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.- And then try to 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 helpful0
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