
Solution retrospective
Hi, I just completed my first project. I would appreciate any feedback and comments to improve my code. Thank you!
Please log in to post a comment
Log in with GitHubCommunity feedback
- @snehamoybag
Hi @hWeitian, Congratulations on completing on your first project 👏 I really liked how you used Flex-box efficiently to match the desired outcome. Great job!
One concern i can notice in your css is the
html { width: 100vw; height: 100vh }
😅 as i've learnt earlier from a fellow frontend mentor dev that setting a fixed width and height can give scroll bars to some device and worse.. the content might even get cropped. So i'd recommend you to never usewidth
to set the pages cause width is by default 100% on all devices and is responsive. On the other hand setting a fixed height can cause the content to overflow or get cropped.. so whenever you need to set a height.. always use themin-height
property 😄👍Marked as helpful
Join 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