Skip to content
Submitted over 3 years ago

Profile Card Component

Tom•245
@tttinh
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Profile card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hello everyone,

Hope you have a good day. This is my 3rd exercise. I would like to receive any feedback or comment to help me improve my skills.

Thanks so much.

Community feedback

  • Vanza Setia•27,775
    @vanzasetiaPosted over 3 years ago

    👋Hi Tom!

    I have some feedback on this solution:

    • Use rem unit instead of px, since it will allow the user change their font size base on the user browser setting.
    • It would be much better if you use ul for the .stats-grid and li for the stats-item

    That's it! Hopefully this is helpful!

    Marked as helpful
    0
    Tom•245
    @tttinhPosted over 3 years ago

    Hi @vanzasetia,

    Thanks for your advice. I am very appreciated! It's really helpful for me.

    Best regards.

    0

Please log in to post a comment

Log in with GitHub

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