Card con html y css para novatos

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.

  • Puede usar la etiqueta <main> para envolver todo el contenido principal de la página en lugar de usar la etiqueta <div>. Con este elemento semántico puedes mejorar la accesibilidad de tu página.

  • 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 to frontendmentor.io.

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

  • Siempre evita saltar niveles de encabezado; Siempre comienza con <h1>, seguido de <h2> y así sucesivamente hasta <h6> (<h1>,<h2>,...,<h6>).

  • Para centrar el componente en la página, debería usar el diseño de Flexbox o Grid. Puede leer más sobre centrado en CSS aquí 📘.

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

  Saludos

  Marked as helpful

  1

  ---

  

  Claudio Bruno•130

  @cdbruno10Posted almost 2 years ago

  @MelvinAguilar Gracias por tus consejos me sirven de mucho, soy muy muy novato y aplicare tus consejos graciasss

  1