Hassia Issah• 50,650
@Hassiai
Posted
In the html replace <div class="container"> with the main tag , there is no need to give the image a div with a class in order to style and put all your text-content in one div:
<main><img><div class=" text"></div></main> To center a content on a page declare the following in the body{ display: flex; align-items: center; justify-content: center; min-height: 100vh} Give the main a background-color and a padding value. For the colors use the color available in the styleguide.md found in the starter folder Give the image a max-width of 100%. Give .text a padding top and bottom value. Hope am helpful Happy Coding.1
Jenrola Jackson• 190
@Majoralter
Posted
@Hassiai Thanks for the feedback. I usually don't code like that I did the challenge late in the night so I didn't take too much notice of these things I'll refactor the code when I get the chance. Also I used all the colors in the style guide.
0