Skip to content
Submitted over 1 year ago

Responsive NFT Card

Paul•100
@ch4r4j4
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

It is the first time that I use "Bootstrap" and "SASS", I know that there are things to improve, thanks for any recommendation

Community feedback

  • Alexandra•580
    @Alexandra2888Posted over 1 year ago

    Hi! Congrats for finishing challenge! Your solution looks very close to the design. Things I think you can improve:

    -in HTML: avoid using internal styles because they will override most other rules as they have higher specificity. Also you can use figure to wrap your image tag for allowing screen readers detect your image. Another tip would be implementing BEM naming convention for classes.

    -in SCSS: implementing BEM naming convention would allow you to nest classes like .main { &__title}} that would lead to a better readability in larger projects. Happy coding!

    Marked 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