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 - Using cursor IDE

Dev Patel 70

@devpatel024

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 specific areas of your project would you like help with?

Gave me feedback!! & if you want to know how to use cursor IDE so message me I will help you to use cursor IDE in your daily coding projects.....

Community feedback

@TedJenkler

Posted

Hi @devpatel024,

I recommend using OG (Open Graph) meta tags due to the shareable nature of your project. These tags will create more attractive links when shared and can also help improve your SEO.

Consider making the QR codes clickable to assist older users who may find it easier to click the link rather than scan the code. Alternatively, you could add a link below the QR code for easy access.

Also, even though this is a small project, I recommend minimizing the use of <div> elements as much as possible in future projects. Replace them with semantic elements where possible. For instance, the card in this project doesn't really need additional <div> elements; it can be structured as a flex column to display all content. This approach will make your code cleaner and be especially helpful in larger projects.

As an extra challenge, try creating your own custom QR code—it's a fun skill to learn and could be a unique touch for your project!

Keep up the great work!

Best, Teodor

Marked as helpful

0

Dev Patel 70

@devpatel024

Posted

Hello @TedJenkler Thank You for you feedback i am know the OG META tags I will update my code shortly.......Thanks Ted.................

1

@EddieBones1

Posted

It looks amazing! but you are missing the line-spacing property for your h1 and p selectors in your stylesheet. That information can be found in the design Figma files. Also, use em instead of px units because px units are not responsive. You can read more about px vs em units by Grace Snow

Marked as helpful

0

Dev Patel 70

@devpatel024

Posted

Hello @EddieBones1 thank you for your suggestions I will apply them soon and update the solution........

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