Completed order summary challenge

Community feedback

• 

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  Freivys Paredes, felicitaciones por acabar este proyecto!

  Revisandolo te recomendaría darle un poco más de tamaño a la cabezera como de 200px para que no salga tan recortada.

  Por otro lado el fondo en tamaño mobile debería ser background-size: contain.

  Te recomendaría agregar transiciones a los botones y enlaces de la siguiente forma transition: all 200ms (Puedes agregarle los valores que quieras). Esto le dará un acabo más profesional al pasar el cursor, o hacer click, etc. a tu proyecto.

  Por cierto corrige los errores que te ha generado el reporte de frontend mentor, por ejemplo has usado src="" alt="" en un div los cuales no son atributos de este ya que son solo para imagenes así que lo mejor sería eliminarlos. Masomenos los errores que salen te dan una idea como corregirlos pero si no sabes me avisas por aquí. Apenas corrijas esos errores puede generar un nuevo reporte para que salga sin ningún error.

  Por lo demás te ha quedado todo bien, buen trabajo :D!

  0

  ---

  

  Freivys Paredes•110

  @FreivysDevPosted over 2 years ago

  @Sdann26 Gracias por esas recomendaciones, aplicare los cambios al código. No me había percate del error con los atributos en el contenedor. 👍

  1