Design comparison
SolutionDesign
Solution retrospective
This is my first webpage with layouts in CSS All feedbacks are welcome
Community feedback
- @ratul0407Posted 11 months ago
@Surya-Thiruvengadam congratulations on completing this challenge🎉🎉
Your solution looks pretty good. But there are room for some changes.
- Always try to use relative css units instead of absolute ones. Pixel(px) is an absolute css unit. And I'd recommend you to avoid this unit completely. Using Pixel for font-sizes is a really bad idea. And I'd suggest not use it for width, height ,margin and padding also. Use rems or ems instead of pixel. If you use pixel in the future while doing large pages with complex section it'll cause a lot of problems and it is also not good for the responsiveness of the page. And if you are wondering why does it matter so much here's an article to explain you more on this topic Why font-sizes should never be in Pixels😄
That was the only nick pick that I found while viewing your solution🙂. Apart from that your solution looks decent
I hope you found this helpful👍👍
Keep up the good work and have a very nice day😄
Marked as helpful0
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