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

Vizo 10

@WiseVizo

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?

flex box:)

Community feedback

@edvelhope

Posted

Ehi there, nice job! The project looks pretty similar! Good responsiveness aswell the code is well-structured and readable. However I would suggest you some improvements: -the background color of the card is slightly different and the qr should have some rounded corners -the text formatting it's not exactly identical -box shadow is missing -in the mobile version the padding change too much -I would suggest to use a tag section to wrap the card

Those are my personal thought on your work, anyway I'm a begginner who wants to improve aswell :)

0
Vyse1986 60

@Vyse1986

Posted

This looks like a great start. I would set the height of the container at 100dvh so the entire height of the page is used when vertically centering the container. Decreasing the width of the container would also help with making it look more like the original design.

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