Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

QR code component

Alejandro 90

@AlejandroLR00

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I have finished this project, it seemed simple to me. Could you give me some comment about my code.

He terminado este proyecto, me pareció sencillo. Me podrían dar algún comentario acerca de mi código.

Community feedback

Fatlind Shehu 2,230

@fatlindshehu

Posted

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

@fatlindshehu

Posted

mark as helpful if you found it helpful!

✌️

0
Danilo Blas 6,300

@Sdann26

Posted

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

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord