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

Varun 10

@worldofvarun

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@chinh1402

Posted

Hello 👋, well done on finishing the project. This is well made, and here are my opinions on your solution.

  1. The description text is a bit too big, it is kind of taking up the focal point of the image, How about trying a smaller font-size
  2. The space between the description text and the bottom edge of the image is too close, the design would look cleaner if you give it more space to work with
  3. A bit nit-picky, but I don't really fond of the pure black colour of headline. You should use the lighter one, probably using the dark blue which was given in the style guide of the challenge

Overall, your solution is good enough. Those above suggestions are just ideas to improve, hope you find them useful

Cheers, and happy coding 🤞

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