QR Code Component

Community feedback

• 

  Nenad Cosovic•360

  @nenadmnePosted almost 2 years ago

  Your solution is very elegant and simple. I loved it =)

  Here are few suggestions:

  * {
      margin:0;
  }
  

  Consider putting bellow margin a global parameter of font-size, and then on rest of the classes use rem as font-size matter. Also putting font-family: 'Outfit', sans-serif; would work great here if its only 1 family for whole page.

  .text h1 {
      font-weight: bold;
  }
  

  Here instead bold you could have used font-weight: 700 (or 600); so it would look same as challenge heading :)

  Great solution by the way =)

  Marked as helpful

  0

  ---

  

  Earl Cardino•90

  @earlyronniePosted almost 2 years ago

  @nenadmne Hi there Nenad!

  Thank you for the helpful tips. I'll make sure to take note of these and apply them to my next projects.

  0