Responsive NFT card using pure SCSS/CSS

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted over 2 years ago

  Hello there! Great work with this one. Regarding the solution you submitted, I think you should know the following notes…

  • For best practice, wrap this line <article class="nft-card"> which is the starting point of the content with main tag. Refer it with this one Semantics
  • Make a separate ruleset for .nft-card__text and add color: var(--soft-blue); or view this line &__image-wrapper &__text in the Sass file if , is needed in between
  • Organized the HTML structure

  Other than that...

  • The project is responsive and utilized the details based on the original design
  • Well done with the hover state of the design, specially the hero image
  • Great job in using Sass for styling, the composition is organized and proper usage of sizing elements
  • The alignment of details and elements are done accordingly

  Above all, The project is awesome! Keep it up!

  1

  ---

  

  Esteban•150

  @ehmenzalaPosted over 2 years ago

  @RioCantre Thank you so much for the feedback! 🖤

  0