Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

qr-code-component-main

@MiguelPositive

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@MelvinAguilar

Posted

¡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

@MiguelPositive

Posted

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

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord