Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive profile card :)

EJā€¢ 170

@nicole-nam

Desktop design screenshot for the Profile card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


feedback is welcome! happy coding :)

Community feedback

Web Frogā€¢ 1,840

@MasterDev333

Posted

@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 :)

0

EJā€¢ 170

@nicole-nam

Posted

@MasterDev333 thank you! that is a good advice!

0
Lukeā€¢ 220

@lukebergmann

Posted

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

0

EJā€¢ 170

@nicole-nam

Posted

@lukebergmann good eye! thank you :)

0

Please log in to post a comment

Log in with GitHub
Discord logo

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