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

Completed order summary challenge

#accessibility#bem#progressive-enhancement#semantic-ui
Freivys Paredes‱ 110

@freivys

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Danilo Blas‱ 6,300

@Sdann26

Posted

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

@freivys

Posted

@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

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