flex

Community feedback

• 

  Jose Osorio Herrera•190

  @Arcanvs

  Posted over 2 years ago

  Muchas felicidades por terminar tu primer desafío, Solo puedo agregar dos recomendaciones para tus futuros desarrollos, utilizar siempre la etiqueta Main como punto de partida para tus desafios y por último puedes instalar una extensión de google chrome 'PerfectPixel' es de gran ayuda para lograr excelentes acabados, no te arrepentirás.

  Happy coding 🐱‍💻

  10
• 

  Melvin Aguilar 🧑🏻‍💻•61,020

  @MelvinAguilar

  Posted over 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.

  • No debería usar CSS en línea porque no es una buena práctica. En su lugar, debe usar una hoja de estilos externa para darle estilo a su página. Al hacer esto, podrá tener una mejor organización del código.

  • Debería usar min-height: 100vh en lugar de height: 100vh. La propiedad height hará que su contenido se corte si el componente es mas grande que la ventana gráfica, puede intentar ver su solución en un dispositivo móvil en modo horizontal y podrá observar como el componente se corta.

  • Todos los estilos del div con clase container son innecesarios, no agrega nada, el elemento body logra centrar completamente el elemento y puede usar el elemento qr-code como la etiqueta <main>

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

  Saludos

  00