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

Media queries

@visheshrao17

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

Ham 340

@hmac100

Posted

Hi Vishesh - I'm by no means an expert but can I suggest using an <h1> tag and a smaller font-size for the .heading class. A smaller font gives 2 lines for the heading instead of 3 so you don't get overflow onto the <p> tag . It also creates more room for left - right padding. The font size within the <p> tag seems smaller in the design jpg. so again a smaller font allows more padding to get nearer to what the design looks like. Also, I could be totally wrong here , but is a class and id actually needed for the <h2> and <p> tag -as this is a super small project ? Any way good work - have fun !

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