Design comparison
Solution retrospective
Hi There! 👋 This is my solution to this challenge 🎉
Any suggestions on how I can improve are welcome 😁
Community feedback
- @vanzasetiaPosted over 2 years ago
Hello, Cosmo! 👋
Congratulations on finishing this challenge! 🎉
It's great that you've put some effort into writing the
README.md
. I like the badges that you have created. Good job on this! 👍The HTML markup is nicely done! You have managed to show those star icons with pure CSS. Amazing! 👏
One thing that I would suggest as far as the HTML is to add
rel="noopener"
to any anchor tags that havetarget="_blank"
. This is a security essential for external links. I suggest reading the web.dev article to learn more about this.On the responsiveness side, I would recommend reducing the amount of
max-width
on themain
element for the mobile layout. Currently, on 600px all the text content is becoming two lines of text which make it harder to read.I hope this helps! Keep up the good work! 👍
Marked as helpful1@cosmoartPosted over 2 years ago@vanzasetia Thank you very much 🙏 I will follow your advice in the next works 😊
0 - @PeteonthebeatPosted about 2 years ago
Congratulations on completing this challenge. As long as I can see, there's pretty much no difference between your solution and the original design!
Keep up the excellent work!
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord