@MrLanter
Posted
Es un buen trabajo y creo que puedo dar algunas sugerencias:
Es importante poner siempre el tamaño del texto en rem y no en px. Las personas amplían el tamaño de su texto en la configuración del navegador y la configuración de px no respeta sus preferencias. Ver aquí para una explicación detallada
Veo que restableciste algunas propiedades predeterminadas del navegador como aquí:
body{
margin: 0;
padding: 0;
}
pero por ejemplo para p y H1 no pudieron estar de acuerdo con el margen y el relleno en 0 especificados en el cuerpo. Podrías usar * para apuntar a todos los elementos. O podrías usar el restablecimiento de CSS.
También puede agregar un cuadro de sombra al .card
como en el modelo. Es más fácil diferenciarlo con el fondo.
Y un último detalle que quizás no te parezca tan inquietante: el desplazamiento innecesario hasta el final de la página. Es un pequeño detalle pero para una página tan pequeña preferimos no tener desplazamiento. Poner estas 2 líneas podría corregir este problema :
.attribution {
position: absolute;
bottom: 20px;
}
Espero que te haya sido útil, que tengas un buen día.
Marked as helpful
@Carinalon
Posted
@MrLanter Muchas gracias por tu aporte, ya me lo guarde para tenerlo en cuenta, siempre es bueno tener este tipo de feedback! gracias!!