Display grid y flex. Background-image y max-width. MediaQuery.

Community feedback

• 

  David•8,000

  @DavidMorgadePosted over 2 years ago

  Buenas Titor, enorabuena por completar el challenge, te quedo bastante bien el layout!

  Me gustaría comentarte un par de cosas para que te puedan ayudar en tus proximos proyectos, con tu permiso por supuesto.

  Primero que todo, el layout está muy bien, pero si quieres mantener todo centrado en la pantalla (me refiero a todo el componente en general, no los elementos de su interior), prueba usando flex en el body en vez de usar margins en el main, podrias quitar esos margins y poner el body así:

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

  Y con eso ya te quedaría todo centrado!

  Otra sugerencia sería que ajustaras un poco el tamaño de los media queries, en vez de hacer el cambio a los 480px (aquí se corta un poco la imagen), yo le dejaría el tamaño de mobile hasta 768px y creo que quedaría mucho mejor, prueba y me comentas!

  Espero que mi feedback te sea de ayuda para tus proximos proyectos, como te he comentado al principio has hecho un muy buen trabajo! Un saludo.

  0