QR card component challenge

Community feedback

• 

  Dan•150

  @igafshellPosted about 2 years ago

  Hey Marcin!

  Congratulations on completing your first challenge on Frontend Mentor!🎉🎉

  I see that you've put much effort into your project and written nice code. I got some tips for you:

  • Use font-size: 15px on the .card, font-weight: 400 on the <p> and font-weight: 700 on the <h1>.
  • Simplify the html structure by removing the .main-screen div, and apply its styling to the body instead. And to keep the footer stuck to the bottom put margin: auto on the .card. This will keep the footer down and center the card at the same time. I advise you to read this article if you want to learn more about that.
  • Add box-content: border-box to the css reset. This will make sizing easier. Read more about it here.
  • This is optional, but you could add some padding, margin and border-radius to the footer to make it more stylish.

  That's about it. Good luck in your next projects.

  Happy Coding!

  Marked as helpful

  0

  ---

  

  Marcin•80

  @mmiskiewicz99Posted about 2 years ago

  @igafshell Thanks a lot!

  1