Design comparison
SolutionDesign
Solution retrospective
All comments are welcome, thank you.
Community feedback
- @ratul0407Posted 11 months ago
@JGabriel19 Your solution is really great🎉🎉
There is not a lot to complain about. But you should remove
.attribute
div that you commented out. And also I would recommend you to start using pixels more. Of course you've used pixels for font size but, padding, border-radiuses and also in a lot of other stuff you should use rems or any relative unit. Relative units are always a better option over absolute units for various reasons.I hope you found this helpful👍
Keep up the good work and have a very nice day😄
1
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