NFT preview card ( Css + Image-Overlay-Icon and Hover Effects)

Community feedback

• 

  Lucas 👾•104,400

  @correlucasPosted about 2 years ago

  👾Hello Achinta, Congratulations on completing this challenge!

  Great solution and great start! By what I saw you’re on the right track. I’ve few suggestions to you that you can consider to add to your code:

  You’ve used px as the unit for sizes but the problem with pixels is that its not optimized for multiple devices and screens. So a good fit its to use rem or em that have a better performance and make your site more accessible between different screen sizes and devices. REM and EM does not just apply to font size, but to all sizes as well.

  ✌️ I hope this helps you and happy coding!

  Marked as helpful

  0

  ---

  

  Achinta Haldar•140

  @FSwebdeveloperPosted about 2 years ago

  @correlucas Thank You for, your useful suggestions to give me provide from your important time to provide your own opinion. Yes, I will apply your suggestion to future projects.

  0
• 

  Hyron•5,870

  @hyrongennikePosted about 2 years ago

  Hi,

  Nice job on the second attempt looks good Just a small tip add the font-family on the body instead of the .card-body because the .card-footer is not in there so it won't get that font that's why the footer font is a different font.

  Marked as helpful

  0

  ---

  

  Achinta Haldar•140

  @FSwebdeveloperPosted about 2 years ago

  Hello ,

  Thank You for , you have notice my little fault to help me correction on this solution and Yes you say right, I can implement on your tip.

  0