margin, padding, box-shadow, border-radius, display, text-align

Community feedback

• 

  Svitlana Suslenkova•5,320

  @SvitlanaSuslenkovaPosted about 2 months ago

  Hi, good job. But you need some corrections. Remove flex from div class=pos and add it to the body (i think you don't need this div at all):

  body { display: flex; flex-direction: column; justify-content: center; align-items: center; min-height: 100vh; } Try this to align(top-bottom) and justify(left-right) your project to the center. It applies to the parent component(body), don't forget about !!min-height!!. You can use grid instead of flex too.

  Hope you found this comment helpful :)

  Marked as helpful

  1
• 

  Wambugu-Francis•30

  @Wambugu-FrancisPosted about 2 months ago

  The solution has semantic HTML and its accessible, has a good layout fits the screen right. The code is clean and precise, can be reused even by a newbie, the solution doesn't differ much from the design.

  1