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

@AmitKumar-001

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


Your feedback is beneficial to me!

Community feedback

Daniel 🛸 44,230

@danielmrz-dev

Posted

Hello @AmitKumar-001

Your project looks great!

Since you asked for feedback, here are some tips for you to improve your project:

  • There's no need for that h3 on top of your page. You can just remove it.

  • Try wrapping your main content in a <main> tag instead of a <div> tag. This makes your HTML more semantic.

  • Every project should have a <h1> tag. We often wrap our main title with it. So your h2 could be a h1, since it's the most important text on the page.

  • If you change your background-color to #D6E1F0, your project will look even closer to the original design.

I hope it helps!

Other than those little details, you did a great job!

Marked as helpful

1

@AmitKumar-001

Posted

Hello @danielmrz-dev

Thank you so much for your feedback. It is very helpful for me to improve my skills. I'm thankful for the suggestion.

1
Daniel 🛸 44,230

@danielmrz-dev

Posted

I'm glad I could help! 😊 @AmitKumar-001

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