@Hassiai
Posted
There is no need to give the body a height and padding value.
To center .main on the page using flexbox or grid instead of margin,
- USING FLEXBOX: add min-height:100vh; display: flex; align-items: center: justify-content: center; to the body
body{
min-height: 100vh;
display: flex;
align-items: center;
justify-content: center;
}
- USING GRID: Add min-height:100vh; display: grid place-items: center to the body
body{
min-height: 100vh;
display: grid;
place-items: center;
}
Reduce the max-width of .main for it to be equivalent to the design. max-width: 320px which is 20rem/em
Give the img a max-width of 100% and a border-radius value for a responsive image, the rest are not needed.
Give h1 and p the same font-size of 15px which is 0.9375rem, text-align: center,the same margin-left, margin-right and margin-top values. Give p a margin bottom value.
Hope am helpful.
Well done for completing this challenge. HAPPY CODING
Marked as helpful
@samoina
Posted
@Hassiai thank you so much for this insightful feedback. Happy to include it in this and other future challenges I do :)