Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

nft-preview-card-component

@AmmarCode

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I'd appreciate it if you can take a look at my solution and give me some feedback. Constructive criticism is what I'm looking for! don't hesitate to be brutally honest :)

Community feedback

Rio Cantre 9,690

@RioCantre

Posted

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!

1
Naveen Gumaste 10,480

@NaveenGumaste

Posted

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

Keep up the good work!👍

1

@AmmarCode

Posted

@Crazimonk Thank you! I appreciate your support!!

0

@khanisrar

Posted

Awesome

1

@AmmarCode

Posted

@khanisrar Thank you!!!

0
Aadvik 1,250

@Aadv1k

Posted

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
1

@AmmarCode

Posted

@Aadv1k Thank you for the great tips, I'll implement that soon.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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