Skip to content
Submitted over 2 years ago

Nft Preview page

Gloryjaw•320
@Gloryjaw
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...NFT preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hii front-end community,

Learned lots of things through making this project( and I thought I knew flex-box really well). Also, I tried to make my html a little more semantic. Any feedback about that is appreciated.

Thanks.

Community feedback

  • Fatlind Shehu•2,230
    @fatlindshehuPosted over 2 years ago

    Hi, Nice work with flex, the component looks great, I have some suggestions for you:

    • Try using rem or em units, and avoid using px, in these projects they're fine, but in bigger ones they can cause unexpected isuses.
    • Try using BEM & SASS in order to make your code more readable & maintainable

    In case you're unfamiliar with BEM and SASS I would suggest watching this tutorial

    Keep it up & happy coding...

    1
    Fatlind Shehu•2,230
    @fatlindshehuPosted over 2 years ago

    In case you found my comment helpful, please mark as helpful

    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