RozangelaPeixoto• 30
@RozangelaPeixoto
Posted
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
@yusadankamu
Posted
@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
