Davisnz• 150
@Davisnz
Posted
Getting there!
Feedback:
-
Don't forget to center the box in the screen, you can do this by using flexbox attributes and adding padding.
-
For the links, you added a div class called "links". Because they will be links, you should use anchors for your links, not divs, so instead of
<div class="links">GitHub</div> you should use:
<div id="links">
<a href="#" class="social-links" >GitHub </a>
<a href="#" class="social-links" >Frontend Mentor </a>
<a href="#" class="social-links" >LinkedIn </a>
<a href="#" class="social-links" >Twitter </a>
<a href="#" class="social-links" >Instagram </a>
</div>
And take it from there :)
Marked as helpful
0
@CarlosArroyoB
Posted
Thanks for the help! I will take it into account for future projects. :)
1
Davisnz• 150
@Davisnz
Posted
@CarlosArroyoB
no problem! I would highly recommend you to fix this one so you can nail the process :)
Marked as helpful
0
