qr-code-component-main

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted about 2 years ago

  Hola @ambioristm 👋, ¡Buen trabajo al completar este desafío! 🎉

  Tengo algunas sugerencias que podría considerar para mejorar su código:

  • Use <footer> en lugar de <div class="attribution">. El elemento <footer> contiene información de autoría.

  • El texto alternativo no debe contener guiones, guiones bajos, ni las palabras "imagen" o "foto", debe ser legible por humanos.

  • Para que los textos alternativos valgan más la pena, agregue texto descriptivo al atributo alt de la imagen QR para explicar qué hace la imagen QR. Al escanear el código QR, será redirigido al sitio web frontendmentor.io, por lo que un ejemplo de texto alternativo sería "QR code to frontendmentor.io". Puede leer más sobre texto alternativo aquí.

  • En lugar de usar píxeles en el tamaño de fuente, use unidades de medida relativas como rem o em. El tamaño de fuente en unidades de longitud absoluta (px) no permite a los usuarios con visión limitada cambiar el tamaño del texto en algunos navegadores. Referencia.

  • La etiqueta <br> no es una etiqueta semántica, por lo que no debe usarla. Además, si un lector de pantalla lee el texto, interrumpirá el flujo de lectura en la etiqueta de salto de línea, por lo que no debe usarse para agregar espaciado vertical. Solo hay algunos casos en los que es necesario (por ejemplo, en un poema o una dirección), y es posible evitarlos aplicando estilos de relleno y margen a través de CSS. Más información aquí.

  Sobre todo, el proyecto está bien hecho👏. ¡Espero que esos consejos te ayuden! 👍

  ¡Buen trabajo! 😁

  1
• 

  Antonio Trinidad Mercedes•70

  @ambioristmPosted about 2 years ago

  Thank you for the feedback

  0