QR Code

Community feedback

• 

  Fatlind Shehu•2,230

  @fatlindshehuPosted over 2 years ago

  Hi @Prathama-Redij,

  This challenge seems easy but sometimes people struggle with it, I can see you did nice work so congrats on that!

  • I can see you centered the component using flex
  • I think this code

  .container {
      max-width: 360px;
      /* margin-top: 10%; */
      /* margin: 0 auto; */
      /* display: flex; */
  }
  

  on the .container is unnecessary ( The commented one)

  • I would suggest using rem or em units, instead of pixel units, px can cause Issues when trying to make your site/component responsive & avoid mixing them (using more than one)

  Keep up the good work!

  Marked as helpful

  0

  ---

  

  Prathama-Redij•30

  @Prathama-RedijPosted over 2 years ago

  @fatlindshehu thanks for suggestion.

  0