Design comparison
Solution retrospective
Gracias por las respuestas, pude solucionar el problema. Cualquier consejo para mejorar, sobre todo los estilos de css es bienvenida, gracias.
Thanks for the answers, I was able to solve the problem. Any advice to improve, especially the css styles is welcome, thanks.
Community feedback
- @DavidMorgadePosted over 2 years ago
Buenas Pato!, enorabuena por finalizar este challenge, te quedo bastante bien y conseguiste que fuera responsive, buen trabajo
Si me permites me gustaría darte algunos consejos que te pueden ayudar.
Primero que todo, intenta centrar el card entero que has creado en medio del body, esto lo puedes hacer facilmente con flexbox, ya le diste la propiedad
min-height: 100vh
, ya solo poniendole flexbox al body y centrando te quedaría perfecto, prueba poniendo tu body así:body { font-family: sans-serif; min-height: 100vh; max-width: 100%; background-color: var(--bg-color); display: flex; justify-content: center; align-items: center; }
Y con eso tendrías todo tu contenido centrado en la pantalla!
Te aconsejo también que le pongas un tag
h1
a tu página, es casi como obligatorio que todas las páginas tengan uno, no pongas más de uno ya que por ello te pueden penalizar los buscadores como Google.Y finalmente te diría que cambies el titulo por algo relacionado en tu
<title>
que tienes en el head, ademas puedes añadir el icono que trae tu challenge con la etiqueta<link>
.Espero que mi feedback te sea de ayuda, cualquier pregunta que tengas no dudes en hacerla, un saludo!
0
Please log in to post a comment
Log in with GitHubJoin 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