Skip to content
Submitted 5 months ago

CSS

RSGames2019•60
@RSGames2019
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...QR code component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Community feedback

  • kaLihaRi•50
    @kalihari90Posted 5 months ago

    Hello. Overall solution is good, but not perfect yet. You forgot to change font-size of the title to the 22px which makes your card component too high. The paragraph below is perfect. Padding seems to be ok everywhere. Try add some box-shadow to the .card, on the bottom especially. Code is readable and well-structured, but if the project will be bigger than this one it would be good practise to use more detailed class names. Good work and keep it going.

    0

Please log in to post a comment

Log in with GitHub

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