HTML y CSS Flexbox

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted almost 2 years ago

  ¡Hola! 👋. ¡Buen trabajo al completar el desafío! Tengo algunos comentarios que podria considerar

  HTML 📄:

  • Usa la etiqueta <footer> para envolver el pie de página en lugar de la etiqueta <div class="attribution">. El elemento <footer> contiene información sobre el autor de la página, los derechos de autor y otra información legal.

  • En este desafío, no se necesita la etiqueta de <picture>, ya que la imagen no necesita cambiar dependiendo del viewport. Esto es necesario con la solución "Product preview card component" porque la plataforma proporciona dos imágenes, una para móvil y otra para escritorio. Puedes usar directamente la etiqueta div en esta solución.

  Alt text 📷:

  • El atributo alt no debe contener las palabras "image", "photo", o "picture", ya que la etiqueta de imagen ya transmite esa información.

  • El atributo alt debe 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.

    Si quieres aprender más sobre el atributo alt, puedes leer este artículo 📘.

  Espero que lo encuentres útil! 😄

  Sobre todo, la solución que has presentado es genial!

  0