QR code component

Community feedback

• 

  Fatlind Shehu•2,230

  @fatlindshehuPosted over 2 years ago

  The challenge is pretty easy, but your approach is great, here’s my opinion about your code:

  • I think you did great using flex to center the component.
  • The elements sizing is perfect.
  • One thing I would bring it up is the usage of px,
  • I think should choose between rem or em units.
  • Avoid using px because in bigger projects it may cause issues while making the site responsive!

  Keep it up & happy coding…

  Marked as helpful

  1

  ---

  

  Fatlind Shehu•2,230

  @fatlindshehuPosted over 2 years ago

  mark as helpful if you found it helpful!

  ✌️

  0
• 

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  Hola Alejandro!

  He revisado tu proyecto y respecto al maquetado te ha quedado 10 de 10, solo pequeños detalles con los tamaños pero te ha quedado en lo personal bastante bien.

  Respecto a tu código, si es que usas visual studio code o algún otro te recomendaría usar la extensión prettier para que formatee tu código y se vea mucho más "bonito", ya que entre etiquetas dejas vacios innecesarios además que es más legible cuando las etiqueas padres enmarcan a las hijas. Como recomendación por mi parte te diría que estudies metodología BEM para el nombramiento de tus clases, hará que todo se más ordenado y sencillo.

  Por ultimo debes eliminar los errores que te presenta el reporte de frontend mentor ya que eliminarlos te ayudará a mejorar bastante, para ellos agregan un <main> encima de <footer> y agrega todas las etiquetas dentro de <div class="card"></div> y esta dentro de <main>. Siempre debes tener un main que contenga todo el contenido principal de tu página una vez por cada página.

  Y con esto listo puedes generar un nuevo reporte y no deberían salirte ningún error.

  Eso seria todo, espero haberte ayudado :D!

  Marked as helpful

  0