Design comparison
Solution retrospective
First ever submit. Any constructive feedback welcome.
Community feedback
- @denieldenPosted over 2 years ago
Hi Eero, great job! Congratulations on completing the challenge.
For remove the scroolbar reset the margin of body:
body { margin: 0; }
Overall you did well :)
Hope this help and happy coding!
Marked as helpful0@eerolliPosted over 2 years ago@denielden Oh yeah, missed the scrollbar completely. Thank you!
1@denieldenPosted over 2 years ago@eerolli You are welcome! I would really appreciate if you mark my comment as helpful if it helped you, thank you very much :)
0 - @2divPosted over 2 years ago
Hi good work.
I am new in field and i was just finish this challenge and i had many issues , but i got tips from the community to improve my practice.
I see you use grid layout and it worked well. However i used flex box layout because its very small requirement and doesn't need much code.
I think the practice its for flex out design.
this is my humble tip. Again good work
0 - @ChamuMutezvaPosted over 2 years ago
Well done. The site looks good
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