Community feedback

• 

  Andrés Gutiérrez Ramírez•2,470

  @AGutierrezRPosted 11 months ago

  Hola! me gusto mucho como ha quedado!

  Tengo algunas sugerencias que podrian interesarte.

  • El body no debe tener limitada su height. En lugar de utilizar height: 100vh, utilice min-height: 100vh.
  • Puedes eliminar los margenes a la izquierda margin-left que usaste en card__header, card__content y .footer y agregar un padding en el .card en si. Esto te ayuda a evitar la microgestion de espacio en el contenido de la tarjeta, ya que en vez de que cada uno sea responsable de espacio, es la tarjeta la responsable.
  • No uses width y height para determinar las dimensiones de la tarjeta, puedes usar un max-width para evitar que crezca mas de la cuenta y el height lo puedes eliminar y dejar que el contenido defina la altura (en este caso es mas util)
  • La ilustracion es de adorno, podrias quitarle el alt que le has colocado, asi lo lectores de pantalla lo ignoraran.
  • El avatar puede beneficiarse de un alt mas descriptivo, como "Headshot of Greg Hooper" o "Avatar of Greg Hooper"

  Espero que esto sea de ayuda 😁. Pero mas importante es decirte que lo estas haciendo muy bien!

  Happy coding!

  Marked as helpful

  0
• 

  Marcos Travaglini•4,840

  @BlackpachamamePosted 11 months ago

  ¡Te quedo muy bueno!

  Te dejo algunos comentarios que pueden ayudarte a mejorar:

  • El <aside class="attribution"> debería ser <footer class="attribution">
  • Acá <a href="#">Your Name Here</a> deberías poner tu nombre
  • En el body repetiste varias propiedades, por ejemplo, pusiste display: flex dos veces
  • En el body la propiedad flex-wrap: nowrap es innecesaria. Podrías agregar un gap: 20px para separar el main del footer
  • Usa min-height: 100vh en lugar de height; 100vh (con esto estas restringiendo una altura especifica, puede generar resultados inesperados)
  • El width: 100vw es innecesario

  Marked as helpful

  0

  ---

  

  TRES•90

  @Tre3s-bytePosted 11 months ago

  @Blackpachamame

  Hola Marcos, muchas gracias por tus comentarios! Lo revisare

  1