@DavidMorgade
Posted
Hello Luciano, congrats on finishing the challenge! pretty good job in my opinion
If I have your permission I would like to give you some advices.
Good job with your semantics I like how you used sections
two separated all the semantic related content, but you could also have used the article
tag in your div class='profile'
, it fits perfectly the semantic of being an article each one of them.
Also don't forget that each section need to have at least one heading, for example, in the names of the profiles you could have used h2 or h3
instead of just a p
.
For the styles I would only advice to increase a bit the size of the general layout cause it looks a bit tiny compared to the solution.
But anyway you did a great job, hope this feedback helps you in your future projects!
Marked as helpful