@DavidMorgade
Posted
Buenas Jorge, muy buen trabajo con CSS y el HTML de tu componente te quedo excelente!
Como consejo te diría que cuando tengas dos items de flex y quieras separarlos (me refiero al main
y el footer
), en vez de utilizar margin-top
o margin
utilices gap
en el elemento padre donde pusiste el flex (en este caso sería en el body
), así que añadiendo gap: 1rem
conseguirias el mismo efecto que usando margin!, esto es un pequeño cambio que apenas influye en nada, pero te lo digo para que lo tengas en cuenta en futuros proyectos!
Además de eso lo unico que cambiaría es tu heading con la clase card__title
, este sería el heading principal por lo tanto deberías de usar directamente un h1
, ya que cualquier página html necesita siempre tener un tag h1
Espero que mi feedback te sea de ayuda, buen trabajo y yo creo que ya deberías empezar a hacer proyectos más complejos!
Marked as helpful
@jlmunozfdev
Posted
@DavidMorgade Otra vez gracias por el feedback. No me había fijado en ese detalle del h1, pensé que lo había puesto. Por otro lado, en cuanto al gap; revisaré un poco la documentación para empaparme más del tema. ¡Gracias nuevamente!