All comments

• 

  I Wayan Dirgayusa• 150

  @yusadankamu

  Submitted 4 months ago

  

  QR code component

  1

  20

  

  RozangelaPeixoto• 30

  @RozangelaPeixoto

  Posted 4 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

  This comment has received 1 replyShow