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
Request path contains unescaped characters
Not Found

Submitted

margin, padding, box-shadow, border-radius, display, text-align

@yatharth023

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

@SvitlanaSuslenkova

Posted

Hi, good job. But you need some corrections. Remove flex from div class=pos and add it to the body (i think you don't need this div at all):

body { display: flex; flex-direction: column; justify-content: center; align-items: center; min-height: 100vh; } Try this to align(top-bottom) and justify(left-right) your project to the center. It applies to the parent component(body), don't forget about !!min-height!!. You can use grid instead of flex too.

Hope you found this comment helpful :)

Marked as helpful

1

@Wambugu-Francis

Posted

The solution has semantic HTML and its accessible, has a good layout fits the screen right. The code is clean and precise, can be reused even by a newbie, the solution doesn't differ much from the design.

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