Fronted Mentor #2

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted over 1 year ago

  ¡Hola! 👋 Excelente trabajo en completar el desafío.

  Tengo otras sugerencias sobre su código que podrían interesarle.

  • El enlace del PREVIEW SITE es incorrecto. Supongo que ha cambiado el nombre del repositorio, pero el enlace muestra un error 404.

  • La etiqueta <figure> debe ser utilizada solo cuando se requieran subtítulos con <figcaption>. En este desafío, no es necesario.

  • Establecer el ancho del componente con un porcentaje o una unidad de viewport se comportará de manera extraña en dispositivos móviles o pantallas grandes. Puede usar un max-width de 320px o 20rem para asegurarse de que el componente tenga un ancho máximo de 320px en cualquier dispositivo. Además, borre la propiedad width con un valor de porcentaje.

  ¡Espero que le resulte útil! 😄 ¡Sobre todo, la solución que envió es genial!

  Saludos

  Marked as helpful

  0

  ---

  

  Dgrl•260

  @DouglasgrlPosted over 1 year ago

  @MelvinAguilar Hola muchas gracias, lo tendre en cuenta estoy comenzando y quiero ir mejorando todos mis errores.

  1
• 

  Francisco Carrillo•5,540

  @frank-itachiPosted over 1 year ago

  Hello there 👋. You did a good job!

  I have some suggestions about your code that might interest you.

  HTML 📄:

  • Wrap the page's whole main content in the <main> tag.
  • If your code has different sections that have a specific purpose like a navigation, article, sections or footer, it’s a good practice to enclose those parts with HTML5 landmarks. For example, you could use a <footer> tag to wrap a footer section.

  I hope you find it useful! 😁😁 Above all, the solution you submitted is great👌!

  Happy <coding /> 😎!

  Marked as helpful

  0

  ---

  

  Dgrl•260

  @DouglasgrlPosted over 1 year ago

  @frank-itachi Muchas gracias , lo tomare en cuenta para los proximos proyectos.!

  0