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

@szramkalukasz

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


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

@GSida015

Posted

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

@akashmishrahaha

Posted

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
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