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 solution

MS2620β€’ 60

@MS2620

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


I wrote this code the way I know, if anyone has any suggestions on what they would change, please let me know so I can learn more.

Community feedback

P
visualdennisβ€’ 8,375

@visualdenniss

Posted

Very nice job! It looks pretty neat, especially on wide screens.

On mobile, starting around 550px and below, the body text seems to be a bit squeezed in. My suggestion to avoid that would be avoiding width: 30% to the card, which takes it its width from its parent(body) and therefore equals to 3rd of the screen size, making it narrow on mobile viewports. You could instead set a max-width: 285px or a size you prefer and make the width: 100% instead or something bigger than the 30%. Hope it helps, keep up the good work!

Marked as helpful

1

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