qr-code-component-main

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted almost 2 years ago

  ¡Hola! 👋. ¡Buen trabajo al completar el desafío! Tengo algunas sugerencias sobre su código que podrían interesarle.

  • Supongo que en el valor del atributo lang trato de poner el idioma español, el valor correcto para español es lang="es". Aun así, toda la página está escrita en inglés y el valor lang debe coincidir con el idioma de toda la página, por lo que un mejor valor sería "english": lang="en"

  • Podría usar la etiqueta <main> para envolver todo el contenido principal de la página en lugar de usar la etiqueta <div>.

  • El atributo alt debería explicar el propósito de la imagen. Al escanear el código QR, el usuario será redirigido al sitio web frontendmentor.io, por lo que un mejor atributo alt sería QR code to frontendmentor.io.

  • El texto Improve Your Front-End Skills by Building Projects se considera un elemento de encabezado (h1).

  Espero que te sea útil! 😄 Por encima de todo, ¡la solución que has presentado es genial!

  Saludos

  Marked as helpful

  1

  ---

  

  Miguel Angel Clavijo Moreno•200

  @MiguelPositivePosted almost 2 years ago

  @MelvinAguilar Muchas gracias por revisar mi código, realizaré las correcciones correspondientes.

  0