Skip to content
Submitted 6 months ago

QR Code

szramkalukasz•20
@szramkalukasz
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?

.

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

.

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

.

Community feedback

  • SIDA•190
    @GSida015Posted 6 months ago

    Hello!👋

    Great work, but I have some comments to make!

    1. Remove the footer from the card! It disrupts your design; you can place it outside of the card.

    2. The border radius of the QR code image is too high; you should reduce it, for example, border-radius: 15px;.

    3. Make sure the width of the card is 320px and the height is 497px!

    4. As you can see, the paragraph of the card is not following the design. You can balance it by reducing its width or adjusting the padding of the card(but I think adjusting with width is right option here). Additionally, consider lowering the font size of the title of your card.

    I hope you find this comment helpful. I would appreciate it if you would mark this comment as helpful.

    Marked as helpful
    1
  • Akash Mishra•160
    @akashmishrahahaPosted 6 months ago

    Hey great work, love the design. But please put the footer outside the design. Hope that helps.

    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