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

@earlyronnie

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


Any feedback is appreciated!

Community feedback

@nenadmne

Posted

Your solution is very elegant and simple. I loved it =)

Here are few suggestions:

* {
    margin:0;
}

Consider putting bellow margin a global parameter of font-size, and then on rest of the classes use rem as font-size matter. Also putting font-family: 'Outfit', sans-serif; would work great here if its only 1 family for whole page.

.text h1 {
    font-weight: bold;
}

Here instead bold you could have used font-weight: 700 (or 600); so it would look same as challenge heading :)

Great solution by the way =)

Marked as helpful

0

@earlyronnie

Posted

@nenadmne Hi there Nenad!

Thank you for the helpful tips. I'll make sure to take note of these and apply them to my next projects.

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