Responsive profile card :)

Community feedback

• 

  Web Frog•1,840

  @MasterDev333Posted almost 4 years ago

  @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-namPosted almost 4 years ago

  @MasterDev333 thank you! that is a good advice!

  0
• 

  Luke•220

  @lukebergmannPosted almost 4 years ago

  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-namPosted almost 4 years ago

  @lukebergmann good eye! thank you :)

  0