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

CodeTag: HTML/CSS QR Creator

Sumaiya92 20

@Sumaiya92

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

@robertdouglasaimon

Posted

Hello budy, how are you? Some tips that can help you! It would be nice if you separated the CSS from the HTML to keep the code clean. It would help you understand where you need to improve in the code.

In the 'body', the 100vh min-height is enough, you just need to adjust the display to flex, align the items in the middle with align-items: center. And here in the body itself, apply the font-size to the entire body as specified by the challenge: 15px.

You named a div in the style, but didn't specify which div it is. And applied several properties to this call, which was for almost all div's (There was a possible conflict). Due to the wrong DIV entry, it would be a good idea to adjust the div entry and the parameter indentation.

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