João Vitor Morita• 210
@joaomorita
Posted
Hey! Great work there! You could use more semantics HTML. You didn't need a container div if you already have a main, you could put the main css at the body tag, and the container css at the main tag. You can use more responsive units to your image and paragraphs too. It's very helpfull look how other people did the same challenge and look for ways to improve. Check out this article about semantc elements: https://www.w3schools.com/html/html5_semantic_elements.asp
I hope I can help you in some way
1
KingX• 150
@iwedibah
Posted
@joaomorita Thank you so much for the review and your correction i will effect it in my next challenge
0
