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

HTML CSS

xatracho 95

@xatracho

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


Any feedback is appreciated

Community feedback

P
AK 6,700

@skyv26

Posted

Hi! xatracho, I just checked you work and you did it good. I found some issues not major one but ofcourse needs to fix it.

  1. You added box-shadow which is really not looking good to eyes. It's okay that you liked it, but keep in mind web development is all about others. Like what are others need and how they react with your solution, because ultimately you made solution in order to ease people lives. Yes, it is not a big project but the little things helps person to grow in a very right way.

  2. You hover states are missing in your design, as it is important to fullfil the requirement.

  3. your design responsiveness is not pretty, I mean your design took whole space of your mobile, tablet view. I think it need to be fix too.

I hope you understand.

Best of luck

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