@Orekihotarou-k
Posted
Hi @mellingh, your solution looks amazing! I especially love how closely it matches the design. Great job🥳🙌!!!
I noticed the links are in separate divs. Using an unordered list <ul>
might be a cleaner way to structure navigation elements, as it's specifically designed for that purpose.
it may look something like this
<ul class="social-links">
<li class="link-item"><a href="#">Github</a></li>
<li class="link-item"><a href="#">Frontend Mentor</a></li>
<li class="link-item"><a href="#">LinkedIn</a></li>
<li class="link-item"><a href="#">Tweeter</a></li>
<li class="link-item"><a href="#">Instagram</a></li>
</ul>
Also, I saw you didn't include box-sizing: border-box
in your reset. It can be helpful for element sizing because it includes padding and borders in the total size. Would you like to know more about it?
Overall, fantastic work! Every project helps us learn and grow as developers. I recorded my solution on YouTube too : KimDoesCode Maybe we can check each other's work out sometime and learn from each other?
Have a wonderful day and happy coding! ✨