Order-summary-component

Community feedback

• 

  David•8,000

  @DavidMorgadePosted about 2 years ago

  Muy buenas, y felicidades por completar este challenge!

  Pocos consejos te puedo dar ya que has estructurado muy bien tu proyecto, he revisado tus tags the html y está todo semanticamente estructurado, muy buen uso del main y de los section.

  Solo cambiaría una cosa de tu proyecto, y es la forma de centrar el componente entero, en vez de usar margin en tu container-main prueba centrarlo desde el body con flex-box, muchas challenges de aquí requieren centrar un componente pequeño en medio de la pantalla, y lo puedes conseguir con pocas lineas de código, el body te quedaría así:

  body {
     height: 100vh;
     display: flex;
     justify-content: center;
     align-items: center;
  }
  

  Con esto ya te quedaría tu componente completamente centrado en la pantalla sin tener que usar margins.

  Te aconsejo que empieces ya con challenge un poco más complicadas ya que está te salió bastante bien.

  Espero que mi feedback te sea de ayuda, cualquier cosa me dices por aquí, buen trabajo!

  Marked as helpful

  0

  ---

  

  FedeSimbron•100

  @FedeSimbronPosted about 2 years ago

  @DavidMorgade muchas gracias, lo tendre en cuenta ya para el proximo challenge

  1