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

Responsive QRCode Card using Flexbox

P
Austin Akersโ€ข 140

@BboyAkers

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


Would love to receive feedback on how should I better structure my Vanilla CSS. Was going to use Sass but decided not to.

Community feedback

@MelvinAguilar

Posted

Hello there ๐Ÿ‘‹. Good job on completing the challenge !

My only suggestion is that you use a CSS reset to eliminate the default styles of the browser. Your solution has around 8px of margin on the body element, causing unnecessary scrolling on desktop devices. Additionally, text elements like paragraphs also have margins both above and below.

I hope you find it useful! ๐Ÿ˜„

Happy coding!

Marked as helpful

1

P
Austin Akersโ€ข 140

@BboyAkers

Posted

@MelvinAguilar Thanks for the feedback!!

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