All comments

• 

  Joe sherbini•20

  @yousefshrpinySubmitted about 3 years ago

  feel free to add any edition or any recommendation for a cleaner code

  

  Frontend-Mentor---Profile-card-component with flexbox

  3

  10
  

  Gia Phong•80

  @giaphong992Posted about 3 years ago

  Great work here mate. But I think you should replace the translate with margin 0 auto to center the div horizontally. For the vertical, you might want to use translate Y(50%) to center the card. However, you did this challenge better than me though. Cheer

  1
• 

  wallyson Ruan•130

  @wallysonruanSubmitted about 3 years ago

  There's a line off the limits of the image, could anyone take a look on my code and tell me what I did wrong?

  

  Pure HTML and CSS, writen on VS CODE | CSS GRID

  3

  10
  

  Gia Phong•80

  @giaphong992Posted about 3 years ago

  Hey bro, great work here. But I notice that you use fixed unit like px too much, so I think you should change and try to use the relative unit. For the problem, it can be solved by using 100% height in img tag and h tag which can remove the line off the limits of the image. Cheers mate, great work though.

  Marked as helpful

  0

  This comment has received 1 replyShow