Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Order-summary-component

@FedeSimbron

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


aprendiendo cada vez mas, cualquier error que encuentren seria de mucha ayuda

Community feedback

David 8,000

@DavidMorgade

Posted

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

Posted

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

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord