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

Quotes make in html, css,js

@SevroAuBarca

Desktop design screenshot for the Advice generator app coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
  • API
2junior
View challenge

Design comparison


SolutionDesign

Community feedback

Danilo Blas• 6,300

@Sdann26

Posted

Daniel, buenas!

Buena solución, un pequeño detalle al boton dale el siguiente atributo display: grid para que quede centrado el icono del dado al medio.

Dejando de lado el diseño ya que te ha queda bastante bien, el tema esta con las semántica de las etiquetas, le falta un main dentro del body, siendo el main el que englobe los demás div para englobar todo el contenido principal del código. Además antes de colocar cualquier h2 o h3 siempre debe ir primero un h1 que de por ley siempre hay uno por página presente o botará errores en este caso para este reto podrías cambiar el h3 por un h1 y ya con clases le cambias los estilos. Por cierto cada cambio que hagas al actualizar tu github revisa el reporte y genera uno nuevo para ver si haz eliminado todos los errores que salgan ahí estos te ayudan a saber si necesitas mejorar alguna que otra cosa y te explican el porque por lo general.

Bueno no hay nada más que agregar buen proyecto y éxito en todo!

Marked as helpful

1

@SevroAuBarca

Posted

@Sdann26 Muchas gracias por el feedback, se agradece mucho, arreglare dichos errores para mejorar mi app. Éxitos para ti también! :D

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