@Hassiai
Posted
Replace <div class="container"> with the main tag and wrap Improve your front-end skills by building project
within <h1>, <h1> Improve your front-end skills by building project </h1>
to make the page/content accessible. click here for more on web-accessibility and semantic html
The body has a wrong background-color. Give it a background-color of hsl(212, 45%, 89%).
There is no need to style .container, to center .box on the page 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;
}
For a responsive content,
- Replace the width of .box max-width, increase its value and replace the height with a padding value for all the sides
max-width: 320px which is 20rem/em padding:16px which is 1rem/em
- Give the img a max-width of 100% and a border-radius value, the rest are not needed.
Give .info a margin value for all the sides, text-align: center and a font-size of 15px which is 0.9375rem, this will be the font-size of both p and h1. Give p a margin-top or h1 a margin-bottom value for the space between the text.
Use relative units like rem or em as unit for the padding, margin, width values and preferably rem for the font-size values, instead of using px which is an absolute unit. For more on CSS units Click here
Hope am helpful.
Well done for completing this challenge. HAPPY CODING
Marked as helpful
@AmroAl-Wageei
Posted
@Hassiai Thank youuu