Componente de tarjeta usando Grid y Flexbox. [BEM]

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted over 1 year ago

  Hola !

  Tengo unos pocos comentarios acerca de tu código.

  • La etiqueta <div> es genial para posicionar y dar estilos a los elementos, pero NO tiene un significado semántico específico. Por eso, te sugiero que evites colocar texto directamente en un <div> y utilices elementos semánticos como párrafos o encabezados.

  • En cuanto a los iconos, Los iconos se pueden considerar como elementos decorativos, así que puedes utilizar un atributo "alt" vacío (alt="") para que los lectores de pantalla los ignoren.

  • No lo escribiste bien y no se hace efecto en este desafío, pero no utilices font-size: 62.5%; para cambiar el tamaño de la letra en el elemento body o html. Te sugiero que leas este artículo en inglés para ver sus desventajas: Should I change the default HTML font-size to 62.5%?

  Espero que estos consejos te sean de ayuda para mejorar la accesibilidad de tus soluciones. Si tienes alguna pregunta, no dudes en hacérmela.

  ¡Saludos cordiales!

  Marked as helpful

  0

  ---

  

  Axel Soto•160

  @axlsotoPosted over 1 year ago

  @MelvinAguilar Muchísimas gracias por los consejos y correcciones, la verdad nunca suelo colocar texto dentro de simplemente un div y ahora realicé esta mala práctica de manera inconsciente. También te agradezco por haberme enviado ese articulo, la verdad esta práctica de cambiar el tamaño de fuente a 62,5% la aprendí en un curso y bueno por suerte la he escrito mal en todos mis códigos y no se ha llegado a aplicar así que no volveré a escribirla en ninguno de mis códigos.

  0