nft-preview-card-component

Community feedback

• 

  Rio Cantre•9,650

  @RioCantre

  Posted over 3 years ago

  Hello there! Good job in this project. You did well in implementing the design and I would like to suggest the following for you...

  • Add the hover state for the text, for example...

  .card-header:hover {
    color: hsl(178, 100%, 50%);
    cursor: pointer;
  }
  
  .creator:hover {
    color: hsl(178, 100%, 50%);
    cursor: pointer;
  }
  
  

  • Same with the hero image... you should add the hover state with the eye icon

  Hope this helps and Keep going!

  10
• 

  Naveen Gumaste•10,420

  @NaveenGumaste

  Posted over 3 years ago

  Hay ! Good Job you made it look nearly perfect to the preview

  Keep up the good work!👍

  11
• 

  Israr khan•60

  @khanisrar

  Posted over 3 years ago

  Awesome

  11
• 

  Aadvik•1,250

  @Aadv1k

  Posted over 3 years ago

  Hey Ammar, congrats on completing your second challenge on frontendmentor! Here are some fixes you should make to better your site.

  • To create the little hover overflow on the image (as seen in the active-states) you can use position absolute to position an element over the image, with opacity set to 0, then on hover you can maybe change the opacity to 0.5, this is how I implemented the hover effect, you can do it by other means if you feel like it.
  • Add a mobile media query to reduce the width of the container and make the fonts a bit smaller. Ciao ~ Aadvik

  11