@mikewil245
Posted
You should follow semantic HTML, for example you have <div class = "links"><a href = "#" class = "links-text">Github</a></div> <div class = "links"><a href = "#" class = "links-text">Frontend Mentor</a></div> <div class = "links"><a href = "#" class = "links-text">Linkedin</a></div> <div class = "links"><a href = "#" class = "links-text">Twitter</a></div> <div class = "links"><a href = "#" class = "links-text">Instagram</a></div>
you could've did <ul><li><a></a></li></ul>
since you have a list of items
Another thing to note is to look into rems and ems instead of using px when using them for setting font-size
, widths
etc..
but besides good job !
Marked as helpful