Skip to content
Submitted 9 months ago

QR Component

VictorGlass•30
@VictorGlass
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...QR code component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

La resolución de problemas que se van dando por medio se va avanzando

What challenges did you encounter, and how did you overcome them?

La verdad al principio se me complico un poco el tema de los margenes pero lo pude resolver. Mas que nada revisando CSS Reference y documentación, ya que hay ocasiones que hasta lo mas sencillo se tiene a olvidar.

What specific areas of your project would you like help with?

mayor comentario en las lineas de código.

Community feedback

  • Adann Jacinto•790
    @DanCodeCraftPosted 9 months ago

    Well done. If anything, I would have it centered on the screen, and not only justified. And maybe mess a little bit with the values in the spacing between the text blocks, so it follows a good element hierarchy, more pleasant to the eyes of a potential customer.

    All the best!

    Marked as helpful
    1
  • Onyinyechi Viola Obika•130
    @ObikaviolaPosted 9 months ago

    Great work 👏🏾💯

    Marked as helpful
    1
  • Chad Elofson•40
    @chadelofsonPosted 9 months ago

    The code looks great. The design looks great. Really the only suggestion is to use more comments in your CSS to organize your code.

    This is a small enough project where it isn't necessary. However, it is a good project to get in the habit to do so.

    Marked as helpful
    0

Please log in to post a comment

Log in with GitHub

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