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

Advice Generator Challenge

#fetch
Pedro Chaparro• 260

@PChaparro

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

Pedro te ha quedado bastante bien el proyecto aunque te recomendaría intentar ser más preciso con los detalles.

Darle un min-heigth a la clase quote hace que se vea mucho espaciado entre la linea que separa la frase y el botón. Te recomendaría en todo caso quitarselo y a la clase quote darle un padding-bottom de 3.5em (Uso em ya que veo más recomendado usar medida relativas que absolutas en este caso.

Entonces te quedaría así:

position: relative;
text-align: center;
font-weight: 800;
width: 100%;
max-width: 640px;
border-radius: 32px;
padding: 32px 42px;
  padding-bottom: 32px;
background-color: var(--neutral-dark-grayish);
padding-bottom: 3.5em;

Además a la paralabra ADVICE yo le pondría el atributo letter-spacing: 4px. Este atributo permite darle separación a las letras y que quede como el diseño.

Bueno eso es lo que te recomendaría mejorar para que quede más fiel al diseño.

Buena suerte :D.

Marked as helpful

1

Pedro Chaparro• 260

@PChaparro

Posted

@Sdann26 Muchas gracias por las recomendaciones, tienes toda la razón, implementaré los cambios!

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