QR code component

Community feedback

• 

  RozangelaPeixoto•30

  @RozangelaPeixotoPosted 6 months ago

  Your solution is great, very similiar to the layout. And I have only a few sugestions:

  • You are loading the font twice. In your style through the import and in your index.html. I suggest you delete the import and left only the links in the head.
  • Another tip would be you remove comments that frontend mentor put in the html and move the internal style to your stylesheet.
  • And finally, if you want this project to be one at your portfolio a good readme file will help you make it more professional and the frontend mentor gives us a great template.

  Good luck with your studies

  Marked as helpful

  0

  ---

  

  I Wayan Dirgayusa•150

  @yusadankamuPosted 6 months ago

  @RozangelaPeixoto Thanks for your feedback; it really came as a surprise, for me, that someone is gonna give me feedback about my work. That means a lot to me as a new frontend programmer. Your input, from the fonts, comment advice, and portfolio, helped me improve and become more professional. Thanks for your constructive input. I greet you from Bali, one of the most beautiful places in Indonesia.

  0