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

página usando HTML e CSS

@israelsobral

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Esse foi o meu primeiro de muitos desafios que irei fazer, gostaria da opinião para saber onde eu posso estar melhorando os códigos.

Community feedback

@Blackpachamame

Posted

¡Buen trabajo Israel!

Algunos consejos:

  • Te aliento a usar flexbox para acomodar elementos de este tipo (card)
  • No usar height: 100vh en la etiqueta html, si no mas bien un min-height: 100vh en el body
  • Para los títulos podrías usar etiquetas como <h1> en lugar de aplicarles la etiqueta <strong></strong>
  • No usar propiedades repetidas para lo mismo, un ejemplo con su proyecto:
p.forte {
    color: black;
    text-align: center;
    padding-top: 30px;
    font-size: 1.4em;
}

p {
    color: rgba(0, 0, 0, 0.558);
    text-align: center;
    padding-top: 30px;
    padding-right: 50px;
}

En este caso, todos los p tienen un text-align: center y un padding-top: 30px, por lo que, no es necesario volver a poner estas propiedades en la clase .forte, a menos que cambie sus valores.

PD: Lo reto a que en su próximo desafío use flexbox CSS.

Marked as helpful

0

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