@danielmrz-dev
Posted
Hello there!
Congrats on completing the challenge! ā
Your solution looks excelent!
I have just one suggestion:
š To improve semantic clarity, try to maintain the titles hierarchy with <h1>
, <h2>
<h3>
, and so on.
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.
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 helpful
@GD-neo
Posted
Thank your for the feed back @danielmrz-dev. Am I always supposed to work 'top down' when using the header elements (<h1> first and then work down the hierarchy)?
My thinking was that <h1>Gabrielle Essence Eau De Parfum</h1>
was the main/most important header on this page which is why I decided to give it the <h1> and make 'Perfume' a <h2>.
Thank you again for the feedback - I hope this question makes some sense.