Responsive profile card :)

Solution retrospective
feedback is welcome! happy coding :)
Please log in to post a comment
Log in with GitHubCommunity feedback
- @MasterDev333
@nicole. Great work!! However, it would be better if you remove scrollbars. You may remove them by setting overflow hidden for the body element. And body tag should have 100vw width and 100vh height. Don't forget overflow-y: auto and remove 100vh in mobile view. Hope it helps. Happy coding :)
- @lukebergmann
Hey Nicole,
Your solution is looking fantastic! Honestly, the only thing that stands out to me is the height and boldness of your 'horizontal rule' above the logistics section. It has an effect that almost looks stuffy and high up. Lowering the hr slightly and making it thinner will give the entire profile card more breathing room!
Happy Coding,
Luke
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