@AdrianoEscarabote
Posted
Hi chilldeleuze, hope you're doing well! I loved how your project turned out, but I’ve got a few suggestions that could be useful:
Using only div
elements, as in the code below, reduces the semantic meaning of the content, which can negatively impact accessibility and SEO. Replacing generic div
tags with more meaningful HTML elements improves clarity for screen readers and search engines.
Improved code with semantic tags:
<section class="profile__container">
<img class="profile__picture" src="./assets/images/avatar-jessica.jpeg" alt="Jessica Randall">
<h2 class="profile__name">Jessica Randall</h2>
<p class="profile__location">London, United Kingdom</p>
<blockquote class="profile__short-bio">"Front-end developer and avid reader."</blockquote>
<nav class="social-links__container">
<a class="social-link__button" href="#">GitHub</a>
<a class="social-link__button" href="#">Frontend Mentor</a>
<a class="social-link__button" href="#">LinkedIn</a>
<a class="social-link__button" href="#">Twitter</a>
<a class="social-link__button" href="#">Instagram</a>
</nav>
</section>
The rest is fantastic.
Hopefully, you'll find it helpful. 👍